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

use File.exist? because Ruby 3.2 does not have File.exists? #15

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

andy-k
Copy link
Contributor

@andy-k andy-k commented Dec 11, 2022

No description provided.

mockdeep added a commit to stringer-rss/stringer that referenced this pull request Dec 31, 2022
It currently [doesn't work with Ruby 3.2][r3]. For our purposes, we can
instead replace it with a small rake task.

[r3]: ku1ik/racksh#15
mockdeep added a commit to stringer-rss/stringer that referenced this pull request Jan 1, 2023
It currently [doesn't work with Ruby 3.2][r3]. For our purposes, we can
instead replace it with a small rake task.

[r3]: ku1ik/racksh#15
dentarg added a commit to spinels/racksh that referenced this pull request Mar 10, 2023
Should have done this before merge of ku1ik#15

Oh well :)
@dentarg
Copy link

dentarg commented Mar 10, 2023

I merged this PR over at https://github.com/spinels/racksh and published spinels-racksh (https://rubygems.org/gems/spinels-racksh)

@rorra
Copy link

rorra commented Jul 4, 2023

hi @sickill , why did the PR didn't get merged? it is a needed update for newer version of ruby and was raised back in 2022... I can help with the gem maintenance if needed

@ku1ik
Copy link
Owner

ku1ik commented Jul 5, 2023

Hey guys. Thanks for this. I don't use Ruby anymore and I don't have many opportunities to maintain this gem or review/test PRs. Let me try to publish this to rubygems right now 🤞

@ku1ik ku1ik merged commit 48c571b into ku1ik:master Jul 5, 2023
@ku1ik
Copy link
Owner

ku1ik commented Jul 5, 2023

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.

4 participants