Fix Style/NegatedIfElseCondition RuboCop offense
What does this MR do and why?
Fixes:
-
Style/NegatedIfElseCondition
RuboCop offense: https://docs.rubocop.org/rubocop/cops_style.html#stylenegatedifelsecondition. -
Style/RedundantFreeze
RuboCop offense: https://docs.rubocop.org/rubocop/cops_style.html#styleredundantfreeze.
Screenshots, screen recordings, or links to review app
Preview of product tier badges:
- (Before): https://docs.gitlab.com/ee/administration/index.html
- (After): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/administration/index.html
- (Before): https://docs.gitlab.com/ee/api/index.html
- (After): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/api/index.html
- (Before): https://docs.gitlab.com/ee/development/index.html
- (After): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/development/index.html
- (Before): https://docs.gitlab.com/ee/subscriptions/index.html
- (After): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/subscriptions/index.html
- (Before - has inline badge): https://docs.gitlab.com/ee/api/project_import_export.html
- (After - has inline badge): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/api/project_import_export.html
Blueprint:
- (Before): https://docs.gitlab.com/ee/architecture/blueprints/composable_codebase_using_rails_engines/index.html
- (After): https://eread-fix-negated-if-else-condition-rubocop-offense.docs.gitlab-review.app/ee/architecture/blueprints/composable_codebase_using_rails_engines/index.html
Merge request acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this merge request.
Edited by Evan Read