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

Update dependency rubocop-shopify to "~> 2.15.0" #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Feb 15, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
rubocop-shopify (source) "~> 2.3.0" -> "~> 2.15.0" age adoption passing confidence

Release Notes

Shopify/ruby-style-guide (rubocop-shopify)

v2.15.1

Compare Source

What's Changed

Bug fixes

Full Changelog: Shopify/ruby-style-guide@v2.15.0...v2.15.1

v2.15.0

Compare Source

What's Changed

Enabled Cops
Changed Cops
  • Naming/InclusiveLanguage's AllowedRegex is changed to permit any unbroken word which includes master. (#​564)
    The motivation for this change is to avoid continuously needing to add exclusions such as mastercard, remaster, etc. to the list, while still allowing the cop to catch problematic uses of the word master.
  • Style/FrozenStringLiteralComment's EnforcedStyle is changed from always to always_true. (#​589)
    Consumers should aim to use frozen_string_literal: true in all files, but if certain files need frozen_string_literal: false, they should disable the cop in those files, mark them as TODO, or add them to the Exclude list.
Changed AllCops Configuration
  • NewCops is changed from enable to disable. (#​581)
    This silences the warning about pending cops for consumers, as the style guide maintainers will triage them instead.
Minimum RuboCop Version

Despite enabling new cops, the minimum RuboCop version is unchanged from the previous release, and remains 1.51.0.

Details

This is possible due to the new approach taken in rubocop.yml, in which ERB is used to conditionally enable cops based on the RuboCop version. (#​582, #​593, #​612)

<% rubocop_version = Gem.loaded_specs.fetch("rubocop").version %>

<% if rubocop_version >= "1.23" %>
Department/NewCopName:
  Enabled: true
<% end %>

The motivation for this is to try the approach ahead of rubocop-shopify including optional configuration for rubocop-* extensions, which would need to work without adding dependencies on the extensions, or their versions.

Relevant PRs

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.14.0...v2.15.0

v2.14.0

Compare Source

What's Changed

Enabled Cops
Changed Cops
  • Tweak Naming/InclusiveLanguage AllowedRegex to allow more git repository links (#​535)
Minimum Rubocop Version

The minimum Rubocop version is now 1.51.0 (https://github.com/Shopify/ruby-style-guide/pull/541).

Relevant PRs

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.13.0...v2.14.0

v2.13.0

Compare Source

What's Changed

Enabled Cops

The following cops have been enabled:

Changed Cops
  • Layout/LineLength now allows inline # rubocop:disable ... comments to exceed the line length. (#​488)
  • Layout/LineLength now allows def test_... lines to exceed the line length limit, making it consistent with test "..." do lines, which were already permitted to exceed the limit. (#​496)
  • Naming/MethodName now allows methods starting with test_ to violate the snake_case convention. (#​511)
  • Style/AndOr now allows and and or outside of conditionals (which must still use && and || instead). (#​515)
Minimum Rubocop Version

The minimum Rubocop version is now 1.50.0 (#​525).

Relevant PRs

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.12.0...v2.13.0

v2.12.0

Compare Source

What's Changed

Enabled Cops

The following cops have been enabled:

Disabled Cops

The following cops have been disabled:

  • Gemspec/DevelopmentDependencies (#​486)
Minimum Rubocop Version

The minimum rubocop version is now 1.44. (#​486)

Full Changelog: Shopify/ruby-style-guide@v2.11.1...v2.12.0

v2.11.1

Compare Source

v2.11.0

Compare Source

What's Changed

Full Changelog: Shopify/ruby-style-guide@v2.10.1...v2.11.0

v2.10.1

Compare Source

Full Changelog: Shopify/ruby-style-guide@v2.10.0...v2.10.1

v2.10.0: 2.10.0

Compare Source

What's Changed

Full Changelog: Shopify/ruby-style-guide@v2.9.0...v2.10.0

v2.9.0: 2.9.0

Compare Source

What's Changed

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.8.0...v2.9.0

v2.8.0: 2.8.0

Compare Source

What's Changed

  • Require Rubocop 1.31.0
  • Enable Lint/ConstantOverwrittenInRescue.

Full Changelog: Shopify/ruby-style-guide@v2.7.1...v2.8.0

v2.7.1: 2.7.1

Compare Source

What's Changed

Non consumer facing change omitted.

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.7.0...v2.7.1

v2.7.0: 2.7.0

Compare Source

What's Changed

New Contributors

Full Changelog: Shopify/ruby-style-guide@v2.6.0...v2.7.0

v2.6.0: 2.6.0

Compare Source

What's Changed

Note that the minimum required Rubocop version has been updated to 1.29.

Full Changelog: Shopify/ruby-style-guide@v2.5.0...v2.6.0

v2.5.0: 2.5.0

Compare Source

What's Changed

Full Changelog: Shopify/ruby-style-guide@v2.4.0...v2.5.0

v2.4.0: 2.4.0

Compare Source

What's Changed

  • Upgraded Rubocop to 1.24
  • Enabled Lint/UselessRuby2Keywords
  • Enabled Style/OpenStructUse

Full Changelog: Shopify/ruby-style-guide@v2.3.0...v2.4.0


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants