Skip to content

(PUP-11993) Style Cops (N-P) #9263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
Feb 28, 2024

Conversation

AriaXLi
Copy link
Contributor

@AriaXLi AriaXLi commented Feb 23, 2024

No description provided.

@AriaXLi AriaXLi requested review from a team as code owners February 23, 2024 22:45
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch from c170b92 to 8a91302 Compare February 23, 2024 22:50
@AriaXLi AriaXLi marked this pull request as draft February 24, 2024 00:09
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch 2 times, most recently from 09502ce to 5a7c7e6 Compare February 26, 2024 22:25
@AriaXLi AriaXLi added the maintenance Maintenance chores are excluded from changelogs label Feb 26, 2024
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch from ef5fa24 to 4553043 Compare February 27, 2024 00:53
@AriaXLi AriaXLi marked this pull request as ready for review February 27, 2024 00:54
@AriaXLi AriaXLi marked this pull request as draft February 27, 2024 01:13
@AriaXLi AriaXLi changed the title (PUP-11993) Style Cops (N-Z) (PUP-11993) Style Cops (N-P) Feb 27, 2024
@AriaXLi AriaXLi marked this pull request as ready for review February 27, 2024 08:12
This commit enables the Style/NegatedIf cop and fixes 152 autocorrectable
offenses.
This commit enables Style/NegatedWhile cop and fixes 6 auto-correctable
offenses.
This commit enables the Style/NestedModifier cop. No offenses were
found.
This commit enables Style/NestedParenthesizedCalls cop and fixes 3
autocorrectable offenses.
This commit enables the Style/NestedTernaryOperator cop and fixes
78 autocorrectable offenses.
This commit enables the Style/Next cop and fixes 152 autocorrectable
offenses.
This commit enables the Style/NilComparison cop and fixes 14 autocorrectable
offenses.
This commit enables the Style/NonNilCheck cop and fixes 4
autocorrectable offenses.
This commit enables the Style/Not cop and fixes 41 autocorrectable
offenses.
This commit enables the Style/NumericLiteralPrefix cop and fixes
74 autocorrectable offenses.
This commit enables the Style/NumericLiterals cop and fixes 19
autocorrectable offenses.
This commit enables the Style/OneLineConditional cop and fixes 3
autocorrectable offenses.
This commit enables the Style/OrAssignment cop and fixes 9
autocorrectable offenses.
This commit enables teh Style/ParallelAssignment cop and fixes
19 autocorrectable offenses.
This commit enables the Style/ParenthesesAroundCondition cop and
fixes 112 autocorrectable offenses.
This commit enables the Style/PercentLiteralDelimiters cop and fixes
121 autocorrectable offenses.
This commit enables the Style/PercentQLiterals cop and fixes 3
autcorrectable offenses.
This commit enables the Style/PerlBackrefs cop and fixes 137
autocorrectable offenses.
This commit enables the Style/Proc cop and fixes 4 autocorrectable
offenses.
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch from af6a26f to 2aa8a61 Compare February 27, 2024 08:17
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch 2 times, most recently from c16fd95 to 476f9bd Compare February 27, 2024 19:51
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch 2 times, most recently from 32ba0ad to d4772b3 Compare February 27, 2024 21:37
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch 2 times, most recently from a884cb2 to 3eda4a4 Compare February 27, 2024 23:24
This commit cleans up some changes Rubocop autocorrected such as
remove extra parenthesis and breaking down ternary operators
into more readable if/else statements.
@AriaXLi AriaXLi force-pushed the PUP-11993/style_cops_N-Z branch from 3eda4a4 to 6b5cb30 Compare February 27, 2024 23:27
@mhashizume mhashizume merged commit 2bc53e6 into puppetlabs:main Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants