[ruby] Update rubocop-rails 2.34.3 → 2.35.0 (minor)#5784
Open
depfu[bot] wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.
What changed?
✳️ rubocop-rails (2.34.3 → 2.35.0) · Repo · Changelog
Release Notes
2.35.0
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 31 commits:
Cut 2.35.0Update ChangelogFix `KeyError` when `shared` key has no `adapter` in `database.yml`Merge pull request #1605 from codergeek121/support-shared-keyMerge pull request #1603 from koic/fix_an_error_in_rails_select_mapFix support for shared key in database.yml[Fix #1602] Fix an error in `Rails/SelectMap`Merge pull request #1597 from 55728/fix-i18n-locale-texts-redirect-back-or-to[Fix #1595] Fix a false negative for `Rails/I18nLocaleTexts` when using `redirect_back_or_to`Merge pull request #1598 from y-yagi/globs_cops_for_testUse glob patterns compatible with Engine or Packwerk for cops targeting `spec/` and `test/` directoriesMerge pull request #1591 from koic/make_rails_strong_parameters_expect_allow_presenceMerge pull request #1592 from lovro-bikic/unknown-env-inequality-comparisonFix false negative for != comparison in Rails/UnknownEnvMake `Rails/StrongParametersExpect` allow `presence`Suppress RuboCop's offensesMerge pull request #1587 from eugeneius/presence_operator_methodsMerge pull request #1586 from eugeneius/presence_parenthesesDon't autocorrect operator methods in Rails/PresenceDon't add unnecessary parentheses in Rails/PresenceMerge pull request #1583 from koic/enhance_rails_strong_parameters_expectMake `Rails/StrongParametersExpect` aware of `params[:key]`Merge pull request #1584 from lovro-bikic/rails-unknown-env-case-supportAdd support for case statements to Rails/UnknownEnvMerge pull request #1582 from lovro-bikic/unknown-env-local-comparisonFix a false negative where 'local' was incorrectly treated as a known environment name when using == comparison in Rails/UnknownEnvDrop `prism_spec` from the default Rake taskMerge pull request #1571 from r7kamura/response-parsed-body-html-methodUse RuboCop RSpec 3.9.0 for developmentMerge pull request #1564 from r7kamura/requires-gem-compatibility-removeReset the docs version✳️ json (2.19.4 → 2.19.5) · Repo · Changelog
Release Notes
2.19.5
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 6 commits:
Release 2.19.5Avoid spamming too many deprecations while parsingTest TruffleRuby release in CI for improved stabilityForce ensure_valid_encoding to be inlined.Use RB_ENC_CODERANGE to first check the cached coderange before calling rb_enc_str_coderange if the coderange is unknown.Fix typo in changelog✳️ rubocop (1.86.1 → 1.86.2) · Repo · Changelog
Release Notes
1.86.2
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 71 commits:
Cut 1.86.2Update ChangelogDon't use Registry#to_hSimplify registry internalsFix false positives in `Style/Copyright`Fix extension suggestion documentation linkMerge pull request #15161 from koic/fix_an_error_for_style_reduce_to_hash.mdMerge pull request #15160 from lovro-bikic/autoload-mixinsAutoload mixins instead of requiring themFix an error for `Style/ReduceToHash` copAutoload correctors instead of requiring themPartially revert auto-gen-config in parallel[Fix #15155] Fix false negatives in `Style/RedundantSelf`Merge pull request #15156 from koic/fix_an_error_for_style_hash_lookup_methodMerge pull request #15154 from deivid-rodriguez/out-with-parallelFix an error for `Style/HashLookupMethod` copFix bug where specifying `--out` disables parallelizationMerge pull request #15151 from bquorning/move-exclude_limit_helperMerge pull request #15150 from koic/fix_false_positives_in_lint_redundant_safe_navigation_for_rescue_and_ensureMerge pull request #15053 from t-daisuke/dos-1stMerge pull request #15149 from koic/fix_false_positives_in_lint_redundant_safe_navigation_for_unless[Fix #15040] Exclude `constants` from `Style/ModuleMemberExistenceCheck`Move shared context "with exclude limit tracking"[Fix #15148] Fix false positives in `Lint/RedundantSafeNavigation`[Fix #15147] Fix false positives in `Lint/RedundantSafeNavigation`Merge pull request #15146 from koic/fix_an_error_in_style_sole_nested_conditional[Fix #15144] Fix an error for `Style/SoleNestedConditional`Merge pull request #15141 from koic/fix_infinite_loop_for_gemspec_require_mfaMerge pull request #15143 from jonas054/15142-fix-infinite-loop[Fix #15142] Fix unbreakable range bug for autocorrectFix incorrect autocorrect for `Gemspec/RequireMFA` causing an infinite loop[Fix #10179] Fix `Style/DocumentDynamicEvalDefinition` false positive on escaped interpolation[Fix #15136] Fix false positive for `Lint/MissingCopEnableDirective` inside `# rubocop:push`/`# rubocop:pop`Merge pull request #15138 from koic/fix_false_positive_warning_for_two_layout_copsMerge pull request #15066 from nicolas-finary/fix/multiline-method-call-indentation-hash-pair[Fix #15137] Fix incorrect "does not support `IndentationWidth` parameter" warning[Fix #15054] Fix false positive for in hash pairs with block receiverMerge pull request #15134 from maxjacobson/mj/update-datetime-docsMerge pull request #14948 from hammadxcm/fix/14821-if-inside-else-comment-autocorrectUpdate Style/DateTime documentationMerge pull request #15132 from koic/fix_incorrect_autocorrect_in_style_copyright[Fix #15130] Fix incorrect autocorrect in `Style/Copyright`[Docs] Fix typosMake `InternalAffairs/LocationLineEqualityComparison` aware of `loc.begin` and `loc.end`Adjust range operation and add autocorrect test in `Lint/UselessAssignment`Merge pull request #15121 from koic/fix_false_positives_in_style_while_until_modifierExtend `Style/WhileUntilModifier` to skip any conditional bodyMerge pull request #15101 from sorafujitani/fix-14971-while-until-modifier-conditional-bodyMerge pull request #15116 from koic/fix_regexp_literal_autocorrect_breaks_syntax_with_curly_braces[Fix #15115] Fix an incorrect autocorrect in `Style/RegexpLiteral`Merge pull request #15112 from koic/fix_false_positive_in_lint_require_relative_self_path[Fix #15107] Fix false positives in `Lint/RequireRelativeSelfPath`Merge pull request #15110 from tdeo/true_runner_parallelism[Fix #15075] Implement true runner parallelismMerge pull request #15109 from alpaca-tc/fix-15106[Fix #15106] Fix TargetFinder to work correctly inside hidden parent directories[Fix #11232] Add --list-enabled-cops-for CLI option[Fix #13755] Run --auto-gen-config in parallelExtract read_exclude_limit spec helper for exclude_limit assertionsMerge pull request #15103 from koic/fix_frozen-error_in_formatter_disabled_config_formatterMerge pull request #15104 from toy/fix-EnforcedStyleForLeadingUnderscores-headerFix space/typo in EnforcedStyleForLeadingUnderscores docs header[Fix #15102] Fix `FrozenError` in `DisabledConfigFormatter`[Fix #14971] Don't suggest modifier form when the body is a conditionalMerge pull request #15100 from jonas054/11828_autocorrect_interference[Fix #11828] Fix brace corrector for EOL commentsMerge pull request #14954 from zeronosu77108/fix_aggregate_template_fragment_autocorrections[Fix #14583] Aggregate template fragment autocorrections per iterationUse AST methods instead of regex to detect spaces in `Lint/ParenthesesAsGroupedExpression`Use AST methods instead of regex to detect empty parentheses in `Style/SymbolProc`Reset the antora docs versionDepfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with
@depfu rebase.All Depfu comment commands