Skip to content

Never break after operator#672

Merged
wvpm merged 1 commit into
devfrom
never_break_after_operator
May 23, 2026
Merged

Never break after operator#672
wvpm merged 1 commit into
devfrom
never_break_after_operator

Conversation

@wvpm
Copy link
Copy Markdown
Contributor

@wvpm wvpm commented Nov 27, 2025

Never break after operators.
Operators are the most important signal of what the code does.
They should always be in sight and never at the end of a line like an afterthought.

@wvpm wvpm added topic:codestyle enhancement New feature or request labels Nov 27, 2025
@wvpm wvpm force-pushed the never_break_after_operator branch 2 times, most recently from a36b1a1 to ba43c46 Compare November 27, 2025 09:58
@wvpm wvpm enabled auto-merge November 27, 2025 10:14
@wvpm wvpm force-pushed the never_break_after_operator branch from ba43c46 to 628e6ef Compare December 7, 2025 18:10
@wvpm wvpm disabled auto-merge December 7, 2025 18:11
@wvpm wvpm enabled auto-merge December 7, 2025 18:23
@wvpm wvpm force-pushed the never_break_after_operator branch from 628e6ef to a69a8ce Compare December 7, 2025 18:24
@wvpm wvpm force-pushed the never_break_after_operator branch from a69a8ce to 711ca4b Compare January 3, 2026 13:06
@zaaarf
Copy link
Copy Markdown
Member

zaaarf commented May 10, 2026

I agree with this, my only complaint is that this PR seems to do a number of things, not all of whom related to the stated task (ex. whatever is going on in main.cpp). Is there a reason it's like that?

@wvpm wvpm force-pushed the never_break_after_operator branch from 711ca4b to 2866390 Compare May 23, 2026 09:58
@wvpm wvpm requested a review from a team as a code owner May 23, 2026 09:58
@wvpm wvpm changed the base branch from master to dev May 23, 2026 09:58
@wvpm wvpm merged commit 8aea9d1 into dev May 23, 2026
13 of 14 checks passed
@wvpm
Copy link
Copy Markdown
Contributor Author

wvpm commented May 23, 2026

Merged to dev as I'm not getting any review

@wvpm wvpm deleted the never_break_after_operator branch May 23, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request topic:codestyle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants