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

ci: test on maintained ruby versions #288

Closed
wants to merge 1 commit into from
Closed

Conversation

igas
Copy link

@igas igas commented Oct 1, 2023

No description provided.

@kou
Copy link
Member

kou commented Oct 1, 2023

Why? We want to support Ruby 2.5.0 or later as much as possible.
https://github.com/ruby/csv/blob/master/csv.gemspec#L57

@igas
Copy link
Author

igas commented Oct 1, 2023

@kou because they are EOL, but I'm happy to just add 3.2. Should I restore 2.x?

@olleolleolle
Copy link
Contributor

👋 Rdoc uses an Action from the https://github.com/ruby/actions repo. It's pretty neat. That way, small edits to CI configuration are often automatic.

@igas
Copy link
Author

igas commented Oct 1, 2023

Nice one @olleolleolle! Happy to switch to it, I guess we just need to agree if we support EOL ruby versions with security vulnerabilities.

@kou
Copy link
Member

kou commented Oct 1, 2023

Should I restore 2.x?

Yes.

Happy to switch to it

I don't object it.

@hsbt hsbt mentioned this pull request Oct 5, 2023
kou pushed a commit that referenced this pull request Oct 5, 2023
This is alternative approach for #288
@kou
Copy link
Member

kou commented Oct 5, 2023

I close this in favor of #289.

@kou kou closed this Oct 5, 2023
@igas igas deleted the patch-1 branch October 6, 2023 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants