Skip to content

Describe base code layout rules#15696

Merged
zverok merged 3 commits into
ruby:masterfrom
zverok:code-layout
Dec 23, 2025
Merged

Describe base code layout rules#15696
zverok merged 3 commits into
ruby:masterfrom
zverok:code-layout

Conversation

@zverok

@zverok zverok commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

While trying to refer to the change of rules for line breaks before the boolean operators, I suddenly discovered that there is no place in docs that describes any line breaking rules. Or maybe I am missing something!

So I tried to sketch a small(ish) document about all that. I contemplated adding something about indentation, too, but for now, I think that the current form of the document might already be helpful?..

@zverok zverok requested review from jeremyevans and nobu December 22, 2025 19:49
@zverok zverok self-assigned this Dec 22, 2025
@zverok zverok added the Documentation Improvements to documentation. label Dec 22, 2025

@jeremyevans jeremyevans left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this. I agree that this information is helpful to document. Only a few minor suggestions.

Comment thread doc/syntax/layout.rdoc Outdated
Comment thread doc/syntax/layout.rdoc Outdated
Comment thread doc/syntax/layout.rdoc Outdated
@zverok zverok requested a review from jeremyevans December 23, 2025 16:50

@jeremyevans jeremyevans left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. This seems good for release. I'm fine with raising this issue after the 4.0 release to see how Matz and other developers want to document these operators.

@zverok zverok merged commit d7d1109 into ruby:master Dec 23, 2025
29 checks passed
@zverok zverok deleted the code-layout branch December 23, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Improvements to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants