Skip to content

Meta: document exceptions to the line-wrapping rule - #12876

Open
zcorpan wants to merge 3 commits into
mainfrom
zcorpan/source-formatting-img
Open

Meta: document exceptions to the line-wrapping rule#12876
zcorpan wants to merge 3 commits into
mainfrom
zcorpan/source-formatting-img

Conversation

@zcorpan

@zcorpan zcorpan commented Sep 1, 2026

Copy link
Copy Markdown
Member

No description provided.

@annevk annevk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine. Might be good to get @domfarolino's agreement given prior disagreements.

Comment thread .github/CONTRIBUTING.md Outdated
#### Line wrapping length

Use a column width of 100 characters and add newlines where whitespace is used. (Emacs, set `fill-column` to `100`; in Vim, set `textwidth` to `100`; and in Sublime, set `wrap_width` to `100`. Alternatively, wrap the paragraph(s) with your changes with https://domenic.github.io/rewrapper/. Make sure that `column length to rewrap` is set to 100.)
Use a column width of 100 characters and add newlines where whitespace is used. (Emacs, set `fill-column` to `100`; in Vim, set `textwidth` to `100`; and in Sublime, set `wrap_width` to `100`. Alternatively, wrap the paragraph(s) with your changes with https://domenic.github.io/rewrapper/ or https://github.com/domfarolino/specfmt/. Make sure that `column length to rewrap` is set to 100.)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should take this opportunity to create nicer looking links for these.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@zcorpan
zcorpan requested a review from domfarolino September 1, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants