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

remove reference to Fixnum (removed in Ruby 3.2) #30

Merged
merged 4 commits into from
Dec 5, 2022
Merged

Conversation

zackse
Copy link
Contributor

@zackse zackse commented Nov 11, 2022

Problems

  • Fixnum is removed from Ruby 3.2+
  • File.exists? is removed from Ruby 3.2+

Solution

  • remove reference and broken check for the use of Fixnum
  • Replace File.exists? with File.exist?
  • bump Ruby requirement for the gem to 2.4+

@zackse zackse force-pushed the ez/ruby-3.2-support branch from 9e37d38 to 820f456 Compare November 11, 2022 19:28
@zackse zackse marked this pull request as ready for review November 11, 2022 19:56
@zackse zackse requested a review from a team as a code owner November 11, 2022 19:56
Gemfile Outdated Show resolved Hide resolved
fittings.gemspec Outdated Show resolved Hide resolved
@zackse zackse force-pushed the ez/ruby-3.2-support branch from 71368e8 to e071985 Compare November 11, 2022 20:38
@zackse zackse merged commit 8276d78 into main Dec 5, 2022
@zackse zackse deleted the ez/ruby-3.2-support branch December 5, 2022 19:25
@@ -179,9 +174,9 @@ def load(params)
# `load` is the behavior we want (in later versions, `load` uses `safe_load`, which doesn't support aliases and
# requires allowlisting classes used in files.
if Psych::VERSION < '3.3.2'
Copy link
Contributor

Choose a reason for hiding this comment

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

what version of Psych comes with Ruby 2.4?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

2.2.2

Copy link
Contributor

Choose a reason for hiding this comment

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

too bad

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