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-shopify from 2.1.0 to 2.4.0 #776

Merged
merged 4 commits into from
Feb 1, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 31, 2022

Bumps rubocop-shopify from 2.1.0 to 2.4.0.

Release notes

Sourced from rubocop-shopify's releases.

2.4.0

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

2.3.0

🍂 🍁 🎃 Happy autumn! 🎃 🍁 🍂

This release includes a change in our update policy. So far we've been very conservative about enabling new cops and rules in the name of not breaking anyone's setups and CI servers. However this also means we've been missing out in a lot of new features and capabilities from the latest versions of RuboCop. That is why from now on we might enable new cops in minor versions of the Ruby Style Guide according to their value and ease of adoption.

For 2.3.0 Pumpkin Spice Latte edition, we've revisited all the cops from the Bundler, Gemspec, and Layout departments and added some additions to our RuboCop config. Most of these changes are autocorrectable. Please try them out and if you have any feedback feel free to open a discussion on GitHub!

Notable changes:

  • Enable Layout cops (#319)
  • Enable Gemspec cops (#317)
  • Enable Bundler cops (#316)
  • Enable Style/ExplicitBlockArgument cop (#314)
  • Add the mastercard brand to AllowedRegex of Naming/InclusiveLanguage (#306)

Stay safe, everyone 👋🏼

2.2.0

  • Use RuboCop 1.18.
  • Disable Lint/ShadowingOuterLocalVariable cop (#273).
  • Enforce consistent comma in Array and Hash literals (#276).
  • Enable Style/InPatternThen and Style/QuotedSymbols cops (#278).
  • Enable Naming/InclusiveLanguage cop (#288).

I'm gonna try to be direct and brief in my release notes so far. You know, everybody's busy these days and I don't want to bother too much just because you want to make sure I'm not breaking your RuboCop CI. Rest assured that I'm trying my best to avoid any breaking changes, promise 😚

Anyway, I hope you and your loved ones are doing well, wherever you are. Coding is great and stuff, but don't forget to disconnect from your screens from time to time. The world is a beautiful place with many possibilities. I recommend also to stay hydrated, exercise, and go for walks in whatever green spaces you have around you. It really makes a difference.

Oh and in case you haven't heard this in a while: you are awesome, just the way you are, and you deserve to be happy. Have a lovely day!

Commits
  • 5bf6961 Prepare for 2.4.0
  • f0c9f96 Require rubocop 1.24
  • 5856e7a Merge pull request #345 from Shopify/dependabot/bundler/rubocop-1.24.1
  • 6807259 Update the configuration for 1.24.1
  • abc2b38 Merge pull request #344 from Shopify/dependabot/npm_and_yarn/eslint-plugin-pr...
  • 93ca4ef Merge pull request #343 from Shopify/dependabot/npm_and_yarn/sass-1.45.2
  • 37335c6 Merge pull request #342 from Shopify/dependabot/github_actions/actions/setup-...
  • 013d3fa Merge pull request #340 from Shopify/dependabot/bundler/minitest-5.15.0
  • d6a2bf2 Bump rubocop from 1.23.0 to 1.24.1
  • 07a288b Bump eslint-plugin-promise from 5.2.0 to 6.0.0
  • 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 dependabot bot added dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code labels Jan 31, 2022
@dependabot dependabot bot requested a review from a team January 31, 2022 21:02
@vinistock
Copy link
Member

To get CI to pass, I had to upgrade i18n since version 1.9.0 was yanked due to this problem. After that, I just auto-corrected the offenses.

@vinistock vinistock force-pushed the dependabot/bundler/rubocop-shopify-2.4.0 branch from 75d9d6c to 60f352c Compare January 31, 2022 21:43
@@ -16,6 +16,10 @@ AllCops:
Include:
- "sorbet/rbi/**/*.rbi"

Naming/InclusiveLanguage:
Exclude:
- "lib/tapioca/compilers/dsl/protobuf.rb"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this because of usage of master?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes - we can't control the URL used in the docs. Also, I can't use the rubocop:disable comment, otherwise that will show up in our docs, so I just turned it off for the file.

Bumps [rubocop-shopify](https://github.com/Shopify/ruby-style-guide) from 2.1.0 to 2.4.0.
- [Release notes](https://github.com/Shopify/ruby-style-guide/releases)
- [Commits](Shopify/ruby-style-guide@v2.1.0...v2.4.0)

---
updated-dependencies:
- dependency-name: rubocop-shopify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@vinistock vinistock force-pushed the dependabot/bundler/rubocop-shopify-2.4.0 branch from 60f352c to 790d306 Compare February 1, 2022 15:29
@vinistock vinistock merged commit 5ac6354 into main Feb 1, 2022
@vinistock vinistock deleted the dependabot/bundler/rubocop-shopify-2.4.0 branch February 1, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants