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.

Sign up for GitHub

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

Jump to bottom

Add rvm-3.107 #4649

Merged
merged 8 commits into from
Apr 1, 2019
Merged

Add rvm-3.107 #4649

merged 8 commits into from
Apr 1, 2019

Conversation

saizai
Copy link
Contributor

@saizai saizai commented Mar 24, 2019

No description provided.

@pkuczynski
Copy link
Member

We dont add md5/sha for rubinius

mpapis
mpapis previously approved these changes Mar 26, 2019
View reviewed changes
Copy link
Member

@mpapis mpapis left a comment

Choose a reason for hiding this comment

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

the config/sha512 should not be added, we were asked by Rubinius team not to store the hashes for Rubinius

otherwise LGTM

@pkuczynski pkuczynski merged commit d5f6757 into rvm:master Apr 1, 2019
@pkuczynski pkuczynski added this to the rvm-1.29.8 milestone Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pkuczynski pkuczynski pkuczynski approved these changes

@mpapis mpapis mpapis left review comments

Assignees
No one assigned
Labels
feature interpreter: rubinius
Projects
None yet
Milestone
rvm-1.29.8
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@saizai @pkuczynski @mpapis