Skip to content

[Fix #12434] Fix a false positive for Lint/LiteralAssignmentInCondition#12438

Merged
koic merged 1 commit intorubocop:masterfrom
koic:fix_a_false_positive_for_lint_literal_assignment_in_condition
Dec 9, 2023
Merged

[Fix #12434] Fix a false positive for Lint/LiteralAssignmentInCondition#12438
koic merged 1 commit intorubocop:masterfrom
koic:fix_a_false_positive_for_lint_literal_assignment_in_condition

Conversation

@koic
Copy link
Member

@koic koic commented Dec 2, 2023

Fixes #12434.

This PR fixes a false positive for Lint/LiteralAssignmentInCondition when using interpolated string or xstring literals.


Before submitting the PR make sure the following are checked:

  • The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
  • Wrote good commit messages.
  • Commit message starts with [Fix #issue-number] (if the related issue exists).
  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
  • Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

…nCondition`

Fixes rubocop#12434.

This PR fixes a false positive for `Lint/LiteralAssignmentInCondition`
when using interpolated string or xstring literals.
@koic koic force-pushed the fix_a_false_positive_for_lint_literal_assignment_in_condition branch from f43dcad to fbe1761 Compare December 2, 2023 08:16
@koic koic merged commit 037c860 into rubocop:master Dec 9, 2023
@koic koic deleted the fix_a_false_positive_for_lint_literal_assignment_in_condition branch December 9, 2023 01: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.

False Positive on Lint/LiteralAssignmentInCondition

1 participant