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

CS: consistent placement of operators #133

Merged
merged 2 commits into from
Mar 22, 2021
Merged

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Mar 22, 2021

Context

  • Code style consistency

Summary

This PR can be summarized in the following changelog entry:

  • Code style consistency

Relevant technical choices:

  • No functional changes.
  • Code style compliance.

CS: consistent placement of boolean operators

Boolean operator should be placed at the start of the line in multi-line conditions

This generally makes for a smaller diff if something changes and improves readability.

CS: concatenation operator at the start of the line

Same thing, but then for multi-line concatenation.

Test instructions

This PR can be tested by following these steps:

  • N/A
    This is a code-only change and should have no effect on the functionality. If the build passes (linting, test runs), we're good.

jrfnl added 2 commits March 22, 2021 07:14
Boolean operator should be placed at the start of the line in multi-line conditions

This generally makes for a smaller diff if something changes and improves readability.
@jrfnl jrfnl added this to the 4.2 milestone Mar 22, 2021
@jrfnl jrfnl merged commit a3dd0c6 into develop Mar 22, 2021
@jrfnl jrfnl deleted the JRF/CS/operator-placement branch March 22, 2021 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant