Skip to content

Comments

Update Ruby major version, remove juweiler dependency#19

Merged
ehimen-io merged 2 commits intomainfrom
owens/2448/ruby-4-upgrade
Feb 9, 2026
Merged

Update Ruby major version, remove juweiler dependency#19
ehimen-io merged 2 commits intomainfrom
owens/2448/ruby-4-upgrade

Conversation

@ehimen-io
Copy link
Contributor

This PR updates Ruby to 4.0.2 and removes the dependency on https://github.com/flajann2/juwelier so we manage the Gem Spec now

@ehimen-io ehimen-io requested review from anero and lavaturtle February 9, 2026 14:38
@ehimen-io ehimen-io added the dependencies Pull requests that update a dependency file label Feb 9, 2026
Copy link
Contributor Author

Can I create a release candidate for this to test with tokyo + any other upstream dependencies?

Now that I ask that, are there any upstream dependencies other than tokyo?

@lavaturtle
Copy link
Contributor

Can I create a release candidate for this to test with tokyo + any other upstream dependencies?

Is there a way to publish that release candidate that doesn't fully publish 4.3.0 to rubygems?

I think the alternative would be: on a branch of tokyo, point at this repo instead of at the published gem.

Now that I ask that, are there any upstream dependencies other than tokyo?

None that I know of! I would be surprised.

Copy link
Contributor

@lavaturtle lavaturtle left a comment

Choose a reason for hiding this comment

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

Nice!

@@ -1 +1 @@
4.2.0 No newline at end of file
4.3.0 No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add a changelog file to this repo, like https://github.com/controlshift/action_kit_rest/blob/main/CHANGELOG.md ?

(We didn't do them in most of these library gems to start, but we've been trying to move towards having them, both because it's nice in general and because then dependabot pulls it in)

spec.add_development_dependency "rubocop"
spec.add_development_dependency "shoulda"
spec.add_development_dependency "simplecov", "= 0.16.1"
spec.add_development_dependency "webmock"
Copy link
Contributor

Choose a reason for hiding this comment

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

does rake need to go in here too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it wasn't there previously so I assume not. I installed locally and integrated with tokyo with no issues (locally!)

s.add_dependency(%q<faraday-detailed_logger>.freeze, ["~> 2.1", ">= 2.1.2"])
s.add_dependency(%q<rubocop>.freeze, [">= 0"])
# Specify which files should be added to the gem when it is released.
# The `git ls-files -z` loads the files in the RubyGem that have been added into git.
Copy link
Contributor

Choose a reason for hiding this comment

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

@ehimen-io ehimen-io merged commit c50b584 into main Feb 9, 2026
2 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants