Skip to content

Conversation

@SixiS
Copy link
Contributor

@SixiS SixiS commented Jun 18, 2025

Marking mutable strings to fix ruby 3.4 warnings.

Copy link
Owner

@iconara iconara left a comment

Choose a reason for hiding this comment

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

Thank you for the fix. I have not made any releases for this gem in years and can't promise that I can do one quickly or even at all, but I will put it on my todo list to make an attempt. Please remove the version bump from the PR, anchor I can merge these changes at least.


module Ione
VERSION = '1.3.0.pre3'.freeze
VERSION = '1.3.1'.freeze
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove this change. The version will be set in a separate commit before a release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for checking it @iconara, I've removed the version change.

@SixiS SixiS force-pushed the ruby_3_4_warnings branch from e665ea3 to f47ae40 Compare June 20, 2025 11:47
@iconara iconara merged commit 233d29e into iconara:master Jun 21, 2025
@iconara
Copy link
Owner

iconara commented Jun 21, 2025

I merged this, but I can't get the tests to run in a modern Ruby. They were written for RSpec 2.x and I guess that isn't working anymore. A lot of the tests seem to require rewriting to get things running, and I don't know how to find the time to do that. I've not written Ruby in years unfortunately.

@SixiS
Copy link
Contributor Author

SixiS commented Jun 24, 2025

@iconara Thanks so much for merging!
Seems the tests just needed a few tweaks to be happy with modern ruby.
#60

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.

2 participants