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 rake Gem to v12 #50908

Merged
merged 2 commits into from Mar 30, 2023
Merged

Update rake Gem to v12 #50908

merged 2 commits into from Mar 30, 2023

Conversation

Hamms
Copy link
Contributor

@Hamms Hamms commented Mar 22, 2023

Specifically to pick up ruby/rake#211, which is required for Rails 6.1 support; without it, we get undefined method ``with_application' for Rake:Module (NoMethodError) when trying to run any rails script.

Specifically, I implemented this by running bundle update rake scss_lint --conservative; scss_lint to pick up the change "Relax gem dependency constraints to allow rake 12.x", and --conservative to avoid also updating the sass gem, just to keep this changeset scoped.

Links

Testing story

Tested locally that this resolves for me the problems with running rake tasks on Rails 6.1

Relying on existing tests to verify that this does not otherwise result in any change to functionality.

Specifically to pick up ruby/rake#211, which is required for Rails 6.1 support; without it, we get `undefined method \`with_application' for Rake:Module (NoMethodError)` when trying to run any rails script.

Note that we specifically implement this by updating the `scss_lint` gem to pick up the change "Relax gem dependency constraints to allow rake 12.x"

- https://github.com/sds/scss-lint/blob/master/CHANGELOG.md#0510
- https://github.com/ruby/rake/releases/tag/v12.2.0
@Hamms Hamms added the Rails Upgrade All work related to upgrading the version of Ruby on Rails we use. label Mar 22, 2023
@Hamms Hamms marked this pull request as ready for review March 27, 2023 18:29
@Hamms Hamms requested a review from a team March 27, 2023 18:29
@Hamms Hamms merged commit 88c2390 into staging Mar 30, 2023
2 checks passed
@Hamms Hamms deleted the rake-12.3.3 branch March 30, 2023 22:08
@Hamms Hamms mentioned this pull request Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rails Upgrade All work related to upgrading the version of Ruby on Rails we use.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants