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

Automatically wrap commit body #56

Merged
merged 1 commit into from
Mar 17, 2022
Merged

Automatically wrap commit body #56

merged 1 commit into from
Mar 17, 2022

Conversation

sonnyp
Copy link
Owner

@sonnyp sonnyp commented Mar 17, 2022

Fixes #52

Defaults to 72 as it seems to be the most popular recommendation. GNOME suggests 75 but 72 will work too.

Here is the rational torvalds/linux#17 (comment) - the fact that we automatically wrap will cause issues with the following

Some things should not be word-wrapped. They may be some kind of
quoted text - long compiler error messages, oops reports, whatever.

But we can add a mode that disable the word wrapping and add a ruler. #58

@sonnyp sonnyp changed the title Automatically wrap the body Automatically wrap commit body Mar 17, 2022
@sonnyp sonnyp force-pushed the wrap-body branch 5 times, most recently from 14b3fd1 to 864dd7a Compare March 17, 2022 21:08
@sonnyp sonnyp marked this pull request as ready for review March 17, 2022 21:08
@sonnyp sonnyp mentioned this pull request Mar 17, 2022
@sonnyp sonnyp force-pushed the wrap-body branch 5 times, most recently from 1230e6b to cd55ef9 Compare March 17, 2022 22:53
@sonnyp sonnyp merged commit 7f571f7 into main Mar 17, 2022
@sonnyp sonnyp deleted the wrap-body branch March 17, 2022 23:08
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

Successfully merging this pull request may close these issues.

Insert hard newlines instead of wrapping long lines
1 participant