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

[Ruby 3.0 support] Array#slice with arithmetic sequence #2526

Merged
merged 8 commits into from
Nov 24, 2021

Conversation

ccocchi
Copy link
Contributor

@ccocchi ccocchi commented Nov 11, 2021

Ruby code isn't very pretty, I tried to match as closely as possible to what CRuby is doing. Also added some tests for edge/weird cases.

@eregon eregon mentioned this pull request Nov 18, 2021
82 tasks
Copy link
Member

@eregon eregon left a comment

Choose a reason for hiding this comment

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

Great work, thank you!
I'm sorry for the late review.

@eregon eregon added the in-ci The PR is being tested in CI. Do not push new commits. label Nov 18, 2021
@eregon eregon added this to the 22.0.0 milestone Nov 18, 2021
@graalvmbot graalvmbot merged commit 424fda8 into oracle:master Nov 24, 2021
@eregon
Copy link
Member

eregon commented Nov 24, 2021

(there was a transient in CI, that's why this took longer to merge)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-ci The PR is being tested in CI. Do not push new commits. oca-signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants