Skip to content

fix(#1030): allow cant- prefix in compound-name lint#1031

Merged
rultor merged 3 commits into
objectionary:masterfrom
ext11n7:1030-cant-prefix
Jul 16, 2026
Merged

fix(#1030): allow cant- prefix in compound-name lint#1031
rultor merged 3 commits into
objectionary:masterfrom
ext11n7:1030-cant-prefix

Conversation

@ext11n7

@ext11n7 ext11n7 commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

This fixes false positives reported in #1030.

Summary:

  • allow the cant- idiomatic prefix in compound-name
  • document the new exception in the motive
  • cover it with the existing idiomatic-prefix regression test

Note: local Maven verification was not available in this environment because mvn is not installed.

@ext11n7

ext11n7 commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

@yegor256

@ext11n7
ext11n7 force-pushed the 1030-cant-prefix branch from ad35f1d to b605130 Compare July 12, 2026 14:45
@volodya-lombrozo

Copy link
Copy Markdown
Member

@rultor merge

@rultor

rultor commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here.

@rultor
rultor merged commit 832a014 into objectionary:master Jul 16, 2026
21 checks passed
@rultor

rultor commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min).

@0crat

0crat commented Jul 16, 2026

Copy link
Copy Markdown

@volodya-lombrozo Thanks for the review! You've earned +4 points for this: +12 as a basis; -8 for absolutely no comments posted; -4 for too few (8) hits-of-code; +4 to give you at least something. Your running score is +461; don't forget to check your Zerocracy account too).

@0crat

0crat commented Jul 16, 2026

Copy link
Copy Markdown

@ext11n7 Thanks for the contribution! You've earned +12 points for this: +16 as a basis; -4 for too few (8) hits-of-code. Please, keep them coming. Your running score is +16; don't forget to check your Zerocracy account too).

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.

4 participants