Skip to content

Support for enforcing vertical layout of enum declarations#1737

Merged
wsafonov merged 2 commits intomaintenance/mps20241from
feature/enums-vertical-formatting
Apr 16, 2026
Merged

Support for enforcing vertical layout of enum declarations#1737
wsafonov merged 2 commits intomaintenance/mps20241from
feature/enums-vertical-formatting

Conversation

@wsafonov
Copy link
Copy Markdown
Collaborator

Fixes #1736

@wsafonov wsafonov requested a review from kbirken April 13, 2026 07:13
kbirken
kbirken previously approved these changes Apr 14, 2026
Copy link
Copy Markdown
Member

@kbirken kbirken left a comment

Choose a reason for hiding this comment

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

Looks good and works nicely, approved. One proposal though: There is a policy for capitalization in descriptions of intentions:

Warning: Naming policies violated: all words except prepositions, articles and particles should be capitalized.

Maybe the new intention should adhere to this policy (although the warning is not displayed because of the .format(...)).

@wsafonov wsafonov force-pushed the feature/enums-vertical-formatting branch from 6f46a92 to 86d2800 Compare April 16, 2026 15:08
@wsafonov
Copy link
Copy Markdown
Collaborator Author

Looks good and works nicely, approved. One proposal though: There is a policy for capitalization in descriptions of intentions:

Warning: Naming policies violated: all words except prepositions, articles and particles should be capitalized.

Maybe the new intention should adhere to this policy (although the warning is not displayed because of the .format(...)).

Thanks for the hint, agree - adjusted the intention name according to the policy.

@wsafonov wsafonov enabled auto-merge April 16, 2026 15:09
Copy link
Copy Markdown
Member

@kbirken kbirken left a comment

Choose a reason for hiding this comment

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

Looks good now, thanks for improving the intention description.

@wsafonov wsafonov merged commit d570564 into maintenance/mps20241 Apr 16, 2026
2 checks passed
@wsafonov wsafonov deleted the feature/enums-vertical-formatting branch April 16, 2026 17:27
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.

2 participants