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 13.0.6 #50909

Merged
merged 2 commits into from Apr 17, 2023
Merged

Update rake Gem to 13.0.6 #50909

merged 2 commits into from Apr 17, 2023

Conversation

Hamms
Copy link
Contributor

@Hamms Hamms commented Mar 22, 2023

Specifically to pick up ruby/rake#326, which is required for Ruby 3 support; without it, we get an error trying to seed scripts:

ArgumentError: wrong number of arguments (given 2, expected 1)
/home/elijah/.rbenv/versions/3.0.5/lib/ruby/3.0.0/fileutils.rb:1137:in `touch'
/home/elijah/.rbenv/versions/3.0.5/lib/ruby/gems/3.0.0/gems/rake-12.3.3/lib/rake/file_utils_ext.rb:35:in `touch'
/home/elijah/projects/work/code-dot-org/dashboard/lib/tasks/seed.rake:153:in `update_scripts'

Specifically, I implemented this by running bundle update rake scss_lint, to also pick up the scss_lint change "Remove dependency on rake gem", which otherwise pins us to rake < 13

Links

Testing story

Tested locally that this resolves for me the problems with seeding on Ruby 3

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.

Specifically, I implemented this by running `bundle update rake scss_lint`, to also pick up the `scss_lint` 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

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.
@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 changed the title Update rake Gem Update rake Gem to 13.0.6 Apr 13, 2023
@Hamms Hamms added Ruby Update Everything related to work to update the version of Ruby our codebase runs on and removed Rails Upgrade All work related to upgrading the version of Ruby on Rails we use. labels Apr 13, 2023
@Hamms Hamms marked this pull request as ready for review April 13, 2023 16:57
@Hamms Hamms requested review from a team April 13, 2023 16:57
@Hamms Hamms merged commit 29056b4 into staging Apr 17, 2023
2 checks passed
@Hamms Hamms deleted the rake-13.0.6 branch April 17, 2023 19:43
@Hamms Hamms mentioned this pull request Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ruby Update Everything related to work to update the version of Ruby our codebase runs on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants