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

Support ruby 3 #249

Closed
wants to merge 1 commit into from
Closed

Support ruby 3 #249

wants to merge 1 commit into from

Conversation

coorasse
Copy link

Adds support for ruby 3

@amit-kando
Copy link

Did the tests passed successfully after those changes?
Can make them pass locally.

@Floppy Floppy mentioned this pull request Apr 1, 2022
@emolayi
Copy link

emolayi commented Apr 28, 2022

Could this PR get merged?

@Floppy
Copy link
Member

Floppy commented Apr 29, 2022

Soon, I hope. I've got a really annoying set of interacting test failures across different ruby versions, and it's being a right pain to solve. Sorry :(

sdt pushed a commit to sdt/csvlint.rb that referenced this pull request Jun 20, 2022
Cherry-picked (by hand) from
Data-Liberation-Front#249
sdt pushed a commit to sdt/csvlint.rb that referenced this pull request Jun 20, 2022
Cherry-picked (by hand) from
Data-Liberation-Front#249
@Floppy
Copy link
Member

Floppy commented Jul 13, 2022

After a lot of wrangling, Ruby 3 support is finally merged in under #264. Thanks for your work here, it helped me get it all straightened out, even though I didn't merge this PR directly!

@Floppy Floppy closed this Jul 13, 2022
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