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.

Sign up for GitHub

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

Jump to bottom

Stop requiring newlines between multiline blocks/expressions #263

Merged
merged 2 commits into from
Dec 12, 2022
Merged

Stop requiring newlines between multiline blocks/expressions #263

merged 2 commits into from
Dec 12, 2022

Commits on Dec 12, 2022

  1. Stop requiring newlines between multiline blocks/expressions 

    This effectively reverts #197. This rule is being removed because we
have found that it forces us to introduce newlines in places where they
obstruct readability rather than helping it.

When grouping statements, it is often useful to "chunk" related lines
together by using a newline before and after the chunk, to denote that
each of the statements are related in some way. This is often done in
unit tests for example, when the "arrange", "act", "assert" grouping
style is used. This rule was forcing us to break up chunks.
    @Gudahtt
    Gudahtt committed Dec 12, 2022
    Configuration menu
    Copy the full SHA
    f278d3e View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into stop-requiring-blank-lines-between-multiline… 

    …-statements
    @Gudahtt
    Gudahtt committed Dec 12, 2022
    Configuration menu
    Copy the full SHA
    cb26bc1 View commit details
    Browse the repository at this point in the history