Skip to content

⬆️ bundler 2.3.15#65

Merged
sunny merged 2 commits into
sunny:mainfrom
pboling:feat/64-fix-deprecation-warnings
Jun 11, 2022
Merged

⬆️ bundler 2.3.15#65
sunny merged 2 commits into
sunny:mainfrom
pboling:feat/64-fix-deprecation-warnings

Conversation

@pboling

@pboling pboling commented Jun 7, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Peter Boling peter.boling@gmail.com

@sunny

sunny commented Jun 7, 2022

Copy link
Copy Markdown
Owner

Since there’s a RuboCop version bump included, can you fix the new issues with it? 🙏🏻

It looks like they can be fixed by simply calling:

bin/rubocop -A

- Fixes sunny#64: deprecation warnings
- Gem updates from `bundle udpate`
- RuboCop linting

Signed-off-by: Peter Boling <peter.boling@gmail.com>
@pboling pboling force-pushed the feat/64-fix-deprecation-warnings branch from 5044c73 to 6c79b9e Compare June 7, 2022 19:09
@pboling

pboling commented Jun 7, 2022

Copy link
Copy Markdown
Contributor Author

@sunny I thought I had already pushed those changes. I need more sleep I think! It should be good now. :)

Comment thread spec/spec_helper.rb Fixed
Comment thread .rubocop.yml
Comment thread .rubocop.yml
Comment thread service_actor.gemspec
Comment thread spec/service_actor/result_spec.rb
@pboling

pboling commented Jun 8, 2022

Copy link
Copy Markdown
Contributor Author

I will bump rubocop-lts to allow it to build against HEAD, which is now Ruby 3.2. That will fix the build. This is the first gem I'm adding rubocop-lts to that I don't maintain myself, so just some minor kinks...

@sunny

sunny commented Jun 8, 2022

Copy link
Copy Markdown
Owner

Wow, I hadn’t encountered these kind of version mixes with RuboCop before. Very happy to use rubocop-lts to help with this!

⬇️ rubocop 1.30.1 => 1.12.1
- Last version to support Ruby 2.4 at runtime

Signed-off-by: Peter Boling <peter.boling@gmail.com>
@pboling pboling force-pushed the feat/64-fix-deprecation-warnings branch from c72283d to e5725cb Compare June 9, 2022 12:02
@pboling

pboling commented Jun 9, 2022

Copy link
Copy Markdown
Contributor Author

@sunny Done!

@sunny

sunny commented Jun 11, 2022

Copy link
Copy Markdown
Owner

Awesome! Great articles about the reasoning behind rubocop-lts, too. Thank you 🎉

@sunny sunny merged commit e0941b7 into sunny:main Jun 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Old bundler & deprecation warnings

3 participants