Skip to content

Conversation

igorpeshansky
Copy link
Member

Copy link

@qingling128 qingling128 left a comment

Choose a reason for hiding this comment

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

LGTM

[Optional] It might be helpful to add comment in the other place to remind future developers to also update checksum in the build containers.

@igorpeshansky
Copy link
Member Author

igorpeshansky commented Mar 31, 2023

Sorry, which other place? This script is not released by us — it's an external tool.

@igorpeshansky igorpeshansky merged commit b0538ca into master Mar 31, 2023
@igorpeshansky igorpeshansky deleted the igorpeshansky-update-rvm branch March 31, 2023 03:53
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.

2 participants