Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Line wrapping #325

Open
bkmgit opened this issue Feb 18, 2021 · 2 comments
Open

Line wrapping #325

bkmgit opened this issue Feb 18, 2021 · 2 comments

Comments

@bkmgit
Copy link
Contributor

bkmgit commented Feb 18, 2021

There is an issue in HPC Carpentry to wrap lines at 79 characters for easier viewing of diffs:
carpentries-incubator/hpc-intro#282
Might this be worth putting in the main lesson-example after testing it out in HPC Carpentry?

@brownsarahm
Copy link
Contributor

maintainer hat on I like this!

I tend to do something roughly like this when I make PRs myself and occasionally will even modify slightly submitted PRs as a maintainer to move toward this, so making it officially a recommendation at least, if not a hard requirement would be nice.

@maxim-belkin
Copy link
Contributor

There is a 100-character limit that is not enforced but checked in a permissive mode via make lesson-check-all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants