Skip to content

Use active voice and improve rule definitions - #8764

Closed
simonschaufi wants to merge 1 commit into
rectorphp:mainfrom
simonschaufi:active-voice
Closed

simonschaufi wants to merge 1 commit into
rectorphp:mainfrom
simonschaufi:active-voice

Conversation

@simonschaufi

Copy link
Copy Markdown
Contributor

Rename:

  • "Changes" to "Change"
  • "Renames" to "Rename"
  • "Removes" to "Remove"

Improve a few rule definitions and apply code style formatting for php methods

@simonschaufi

Copy link
Copy Markdown
Contributor Author

The failing test is not related to this PR!

Rename:
- "Changes" to "Change"
- "Renames" to "Rename"
- "Removes" to "Remove"
- "Shortens" to "Shorten"

Improve a few rule definitions and apply code style formatting for php methods
@TomasVotruba

Copy link
Copy Markdown
Member

I'd prefer to keep it as it it, pretty clear what rule does.

@simonschaufi

simonschaufi commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor Author

Yes, I agree with you that it is clear but some messages are in active voice and some are in "this rule does this" voice. You may not see a big benefit but for me this is really important. I have a tool where I extract the rule definitions and I need consistency there so I ask you kindly to reopen this PR and do me a favor. Additionally your docs rendering improves as it prints code nicely on your website 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants