Update rubocop-rails 2.29.1 → 2.30.2 (minor) #1423
Closed
Conversation
This file contains 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
![depfu[bot]](https://avatars.githubusercontent.com/in/715?s=60&v=4){kind=small}
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.29.1 → 2.30.2) · Repo · Changelog
Release Notes
2.30.2
2.30.1
2.30.0
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 41 commits:
Cut 2.30.2Update ChangelogMerge pull request #1450 from vlad-pisanov/vp_fix_lexically_scoped_action_filter_2Merge pull request #1428 from Earlopain/uniq-pluck-multiple-linesFix incorrect comment placement for `add_dependency`[Fix #1427] Fix an error for `Rails/UniqBeforePluck` with multi-line expression[Fix #1447] Fix false positive for `Rails/LexicallyScopedActionFilter` when action methods are delegatedEnable `InternalAffairs/CopEnabled` copMerge pull request #1448 from ydakuka/1446/fix-incorrect-autocorrect-for-rails-delegate-when-module_function-is-used[Fix rubocop#1446] Fix false positives for `Rails/Delegate` when `module_function` is used.Switch back docs version to masterCut 2.30.1Update ChangelogMerge pull request #1445 from koic/fix_an_error_when_using_target_rails_versionMerge pull request #1444 from koic/fix_an_incorrect_behavior_when_using_migrated_schema_version[Fix #1442] Fix an incorrect behavior when using `AllCops: TargetRailsVersion`Fix an incorrect behavior when using `AllCops: MigratedSchemaVersion`Switch back docs version to masterRestore useful commentCut 2.30.0Update ChangelogUse RuboCop Performance 1.24 for developmentMerge pull request #1434 from koic/pluginfy_with_lint_rollerPluginfy RuboCop RailsMerge pull request #1441 from koic/fix_an_error_for_rails_delegate_copFix an error for `Rails/Delegate`Merge pull request #1438 from ydakuka/912-add-more-delegation-targets-to-rails-delegateSuppress RuboCop's offense[Fix rubocop#912] Enhance `Rails/Delegate` by adding delegation detection for `self.class`, constants, instance variables, and class variablesMerge pull request #1433 from ydakuka/1071-fix-an-error-occurring-in-the-rails-file-path-cop-when-file-join-is-used-with-a-variable[Fix rubocop#1071] Fix `Rails/FilePath` cop to correctly handle `File.join` with variables and ignore leading and multiple slashes in string literal arguments for `Rails.root.join` and `File.join`Suppress redundant configuration logging for rubocop-railsAvoid unnecessary `send`Suppress RuboCop's offensesAdd `shared_context` for Rails 8.1Merge pull request #1432 from dvandersluis/use-node-groupsUse node groups in node patterns to replace unions of typesUse `Node#any_block_type?`Fix a build errorRemove a redundant config in spec_helper.rbSwitch back docs version to master✳️ rubocop (1.71.2 → 1.72.2) · Repo · Changelog
Release Notes
1.72.2
1.72.1
1.72.0
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 72 commits:
Cut 1.72.2Update ChangelogCorrect Style/RedundantParentheses documentation[Fix #13861] Fix `ArgumentError` related to two deprecated `AllowedPattern` APIs[Fix #13849] Fix an error for `Lint/UselessConstantScoping`Fix an error for `Style/RedundantFormat`[Fix #13856] Fix false positives for `Lint/UselessConstantScoping`[Fix #13844] Fix an error for `Style/RedundantFormat`Fix exclusion of relative paths in plugin's `AllCops: Exclude` as expected[Fix #13850] Fix false negatives for `InternalAffairs/ExampleDescription`[Fix #13846] Mark `Style/RedundantFormat` as unsafe autocorrectUse RuboCop RSpec 3.5 for developmentFix documentation for `Naming/BlockForwarding` and `Style/ArgumentsForwarding`[Doc] Tweak an example in the Plugin Migration GuideUse RuboCop Rake 0.7.0 for developmentMerge pull request #13842 from koic/use_rubocop_performance_1_24Use plugified RuboCop Performance (1.24) for developmentReset the docs versionCut 1.72.1Update ChangelogMake plugin load error messages more user-friendly[Fix #13839] Fix false positives for `Lint/RedundantTypeConversion`Extension plugin is loaded automatically with `require 'rubocop/rspec/support'`[Fix #13836] Fix an error for `Style/RedundantParentheses`Reset docs versionCut 1.72Update ChangelogAllow `RuboCop::ConfigLoader.inject_defaults!` to accept `Pathname` instancesAdd new `Lint/UselessConstantScoping` copMerge pull request #13829 from dvandersluis/redundant-format-literalsUpdate `Style/RedundantFormat` to register an offense for `format` arguments that can be easily inlinedMerge pull request #13824 from koic/fix_incorrect_plugin_exception_nameMerge pull request #13822 from Shopify/vs-fix-undefined-method-loggerFix undefined method Logger when processing watched file notificationsFix incorrect plugin exception nameFix unexpected removal of plugin configuration valueMerge pull request #13817 from dvandersluis/format-parameter-mismatch-bugRemove useless `File.join`Fix false positive for format specifier with non-numeric precision[Fix #12672] Fix false positives for `Lint/FormatParameterMismatch` when the width value is interpolated[Fix #13661] Make server mode detect local paths in `inherit_from` and `require`Suppress `warning: fiddle/import is found in fiddle`Merge pull request #13815 from koic/make_suggest_extensions_aware_of_plugin_extensionsMake `SuggestExtensions` aware of plugin extensions[Fix #13811] Fix false negatives for `Style/RedundantParentheses`[Fix #13800] Add new `Lint/SuppressedExceptionInNumberConversion` cop[Fix #13808] Fix false negatives for `Style/RedundantParentheses`Make the language_server-protocol dependency version stricterMerge pull request #13809 from koic/fix_false_negative_for_style_redundant_parentheses[Fix #13807] Fix false negatives for `Style/RedundantParentheses`Centralize `RuboCop::LSP.enable` activation and rename adapter classMake the lint_roller dependency version stricter[Fix #6012] Support RuboCop extension pluginMerge pull request #13799 from dvandersluis/issue/12795[Fix #12795] Fix `Layout/BlockAlignment` for blocks that are the body of an endless methodMerge pull request #13798 from dvandersluis/issue/11655[Fix #11655] Add `--ignore-disable-comments` to Basic Usage documentationMerge pull request #13797 from koic/remove_useless_suppressed_webmock_warningRemove useless suppressed WebMock warningAdd new `Lint/RedundantLiteralConversion` copNaming/PredicateName: Optionally use Sorbet to detect predicate methods (#13721)Fix `Style/RedundantFormat` offensesAdd new `Style/RedundantFormat` cop to check for uses of `format` or `sprintf` with only a single string argumentMerge pull request #13796 from dvandersluis/fix/empty-lines-around-method-body-endlessFix crash in `Layout/EmptyLinesAroundMethodBody` for endless methodsSuppress redundant default configuration logging from `inject_defaults!`Suppress redundant default configuration logging from `InternalAffairs/UndefinedConfig`Suppress redundant configuration logging for internal affairsApply `InternalAffairs/LocationExists` correctionsAdd new `InternalAffairs/LocationExists` cop to check for code that can be replaced with `Node#loc?` or `Node#loc_is?`Add new `Lint/CopDirectiveSyntax` copReset the docs versionRelease Notes
2.10.1 (from changelog)
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 4 commits:
Release 2.10.1Merge pull request #749 from byroot/fix-state-roundtripFix a compatibility issue with `MultiJson.dump(obj, pretty: true)`Update changelogSorry, we couldn't find anything useful about this release.
Commits
See the full diff on Github. The new version differs by 2 commits:
Bump patch version.Escape non-printable characters when logging.🆕 lint_roller (added, 1.1.0)
Depfu 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