Skip to content
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

Bump rubocop-rails from 2.6.0 to 2.9.1 #4288

Merged
merged 6 commits into from
Aug 10, 2021

Conversation

dependabot-preview[bot]
Copy link
Contributor

@dependabot-preview dependabot-preview bot commented Dec 16, 2020

Bumps rubocop-rails from 2.6.0 to 2.9.1.

Release notes

Sourced from rubocop-rails's releases.

RuboCop Rails 2.9.1

Bug fixes

  • #408: Fix bug in Rails/FindEach where config was ignored. (@ghiculescu)
  • #401: Fix an error for Rails/WhereEquals using only named placeholder template without replacement argument. (@koic)

Changes

  • #404: Make Rails/HelperInstanceVariable accepts of instance variables when a class which inherits ActionView::Helpers::FormBuilder. (@koic)
  • #406: Deconstruct "table.column" in Rails/WhereEquals. (@mobilutz)

RuboCop Rails 2.9.0

New features

  • #362: Add new Rails/WhereEquals cop. (@eugeneius)
  • #339: Add new Rails/AttributeDefaultBlockValue cop. (@cilim)
  • #344: Add new Rails/ArelStar cop which checks for quoted literal asterisks in arel_table calls. (@flanger001)
  • #389: Add IgnoredMethods config option for Rails/FindEach cop. (@tejasbubane)

Bug fixes

  • #371: Fix an infinite loop error for Rails/ActiveRecordCallbacksOrder when callbacks have inline comments. (@fatkodima)
  • #364: Fix a problem that Rails/UniqueValidationWithoutIndex doesn't work in classes defined with compact style. (@sinsoku)
  • #384: Mark unsafe for Rails/NegateInclude. (@koic)
  • #394: Fix false offense detection of Rails/RedundantAllowNil when using both allow_nil and allow_blank on different helpers of the same validator`. (@ngouy)

Changes

  • #383: Require RuboCop 0.90 or higher. (@koic)
  • #365: Mark Rails/SquishedSQLHeredocs unsafe for autocorrection. (@tejasbubane)

RuboCop Rails 2.8.1

Bug fixes

  • #345: Fix error of Rails/AfterCommitOverride on after_commit with a lambda. ([@pocke][])
  • #349: Fix errors of Rails/UniqueValidationWithoutIndex. ([@Tietew][])
  • #338: Fix a false positive for Rails/IndexBy and Rails/IndexWith when the each_with_object hash is used in the transformed key or value. (@eugeneius)
Changelog

Sourced from rubocop-rails's changelog.

2.9.1 (2020-12-16)

Bug fixes

  • #408: Fix bug in Rails/FindEach where config was ignored. ([@ghiculescu][])
  • #401: Fix an error for Rails/WhereEquals using only named placeholder template without replacement argument. ([@koic][])

Changes

  • #404: Make Rails/HelperInstanceVariable accepts of instance variables when a class which inherits ActionView::Helpers::FormBuilder. ([@koic][])
  • #406: Deconstruct "table.column" in Rails/WhereEquals. ([@mobilutz][])

2.9.0 (2020-12-09)

New features

  • #362: Add new Rails/WhereEquals cop. ([@eugeneius][])
  • #339: Add new Rails/AttributeDefaultBlockValue cop. ([@cilim][])
  • #344: Add new Rails/ArelStar cop which checks for quoted literal asterisks in arel_table calls. ([@flanger001][])
  • #389: Add IgnoredMethods config option for Rails/FindEach cop. ([@tejasbubane][])

Bug fixes

  • #371: Fix an infinite loop error for Rails/ActiveRecordCallbacksOrder when callbacks have inline comments. ([@fatkodima][])
  • #364: Fix a problem that Rails/UniqueValidationWithoutIndex doesn't work in classes defined with compact style. ([@sinsoku][])
  • #384: Mark unsafe for Rails/NegateInclude. ([@koic][])
  • #394: Fix false offense detection of Rails/RedundantAllowNil when using both allow_nil and allow_blank on different helpers of the same validator`. ([@ngouy][])

Changes

  • #383: Require RuboCop 0.90 or higher. ([@koic][])
  • #365: Mark Rails/SquishedSQLHeredocs unsafe for autocorrection. ([@tejasbubane][])

2.8.1 (2020-09-16)

Bug fixes

  • #345: Fix error of Rails/AfterCommitOverride on after_commit with a lambda. ([@pocke][])
  • #349: Fix errors of Rails/UniqueValidationWithoutIndex. ([@Tietew][])
  • #338: Fix a false positive for Rails/IndexBy and Rails/IndexWith when the each_with_object hash is used in the transformed key or value. ([@eugeneius][])
  • #351: Add <> operator to Rails/WhereNot cop. ([@Tietew][])
  • #352: Do not register offense if given a splatted hash. ([@dvandersluis][])
  • #346: Fix a false positive for Rails/DynamicFindBy when any of the arguments are splat argument. ([@koic][])
  • #357: Fix a false positive for Rails/ReversibleMigration when keyword arguments of change_column_default are in the order of to, from. ([@koic][])

2.8.0 (2020-09-04)

New features

Commits
  • addc9d3 Cut 2.9.1
  • 8d10c49 Merge pull request #406 from mobilutz/patch-1
  • 8e4425c Rails/WhereEquals add table/column split
  • 0728968 Merge pull request #407 from koic/make_rails_helper_instance_variable_accept_...
  • 4741d6b [Fix #404] Make Rails/HelperInstanceVariable accept of instance variables
  • 97c8578 Merge pull request #405 from koic/fix_an_error_for_rails_where_equals
  • 3eecd98 [Fix #401] Fix an error for Rails/WhereEquals
  • abbb4fa Merge pull request #408 from ghiculescu/find-each-bug
  • 0ffda34 Fix bug in Rails/FindEach
  • 151b8a6 Fix a build error
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

@dependabot-preview dependabot-preview bot added the dependencies Pull requests that updates a dependency label Dec 16, 2020
@dependabot-preview dependabot-preview bot force-pushed the dependabot/bundler/rubocop-rails-2.9.1 branch 2 times, most recently from 8502179 to d1e8fd8 Compare March 9, 2021 15:39
@dependabot-preview dependabot-preview bot force-pushed the dependabot/bundler/rubocop-rails-2.9.1 branch from d1e8fd8 to 56abcf2 Compare March 20, 2021 14:31
@dependabot-preview
Copy link
Contributor Author

This pull request will no longer be automatically closed when a new version is found as this pull request was created by Dependabot Preview and this repo is using a version: 2 config file. You can close this pull request and let Dependabot re-create it the next time it checks for updates.

@javierm javierm force-pushed the dependabot/bundler/rubocop-rails-2.9.1 branch from 56abcf2 to 65b254f Compare August 9, 2021 20:00
@javierm javierm force-pushed the dependabot/bundler/rubocop-rails-2.9.1 branch 3 times, most recently from f458b36 to 3865225 Compare August 9, 2021 21:51
We were already following it most of the time.
We simplify the code a little bit, and make it more consistent since we
were already using `where.not` in most places.
We were already following this style in most places.
These issues were detected after upgrading to rubocop-rails >= 2.8.0.
@javierm javierm self-assigned this Aug 9, 2021
@javierm javierm added this to Reviewing in Consul Democracy via automation Aug 9, 2021
Consul Democracy automation moved this from Reviewing to Testing Aug 10, 2021
@javierm javierm merged commit ab9e99f into master Aug 10, 2021
Consul Democracy automation moved this from Testing to Release 1.4.0 Aug 10, 2021
@javierm javierm deleted the dependabot/bundler/rubocop-rails-2.9.1 branch August 10, 2021 11:23
@javierm javierm added the Linters Rubocop, ERB Lint, ESLint, SCSS-Lint, ... label Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that updates a dependency Linters Rubocop, ERB Lint, ESLint, SCSS-Lint, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants