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

the balanced line #20

Merged
merged 3 commits into from
Dec 20, 2023
Merged

the balanced line #20

merged 3 commits into from
Dec 20, 2023

Conversation

arnetheduck
Copy link
Owner

@arnetheduck arnetheduck commented Dec 20, 2023

Balance newlines in various places:

  • in sections (var, type etc), use long-form when contents span multiple lines already
  • in if/statement expressions, use compact single-line format when it fits
  • in long procs with pragmas, break parameters early if pragma doesn't fit

Balance newlines in various places:

* in sections (var, type etc), use long-form when contents span multiple
lines already
* in if/statement expressions, use compact single-line format when it
fits
this typically puts the pragma on its own line together with the return
type which doesn't add lines but makes the parameters more visible
@arnetheduck arnetheduck merged commit 0f26a0a into master Dec 20, 2023
6 checks passed
@arnetheduck arnetheduck deleted the lines branch December 20, 2023 15:07
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.

1 participant