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

feat: Adding syncing of NNS released wasm versions #3148

Merged
merged 40 commits into from
Dec 18, 2024

Conversation

NikolaMilosa
Copy link
Contributor

@NikolaMilosa NikolaMilosa commented Dec 12, 2024

This PR refactors the script used for updating mainnet revisions in testnets and adds the ability for it to be used more robustly while preserving the existing features.

On top of that, this PR extends the update-mainnet-revisions workflow to add one more job that will be run in parallel and will update the mainnet-canisters.json file.

Example of new successful run: https://github.com/dfinity/ic/actions/runs/12297897124
Example of created PR: #3198

@github-actions github-actions bot added the feat label Dec 12, 2024
@NikolaMilosa NikolaMilosa marked this pull request as ready for review December 12, 2024 14:00
@NikolaMilosa NikolaMilosa requested a review from a team as a code owner December 12, 2024 14:00
@github-actions github-actions bot added the @idx label Dec 12, 2024
@sasa-tomic sasa-tomic changed the title feat: Adding syncing of nevrous system released wasms feat: Adding syncing of NNS released wasms Dec 12, 2024
@sasa-tomic sasa-tomic changed the title feat: Adding syncing of NNS released wasms feat: Adding syncing of NNS released wasm versions Dec 12, 2024
@aterga
Copy link
Member

aterga commented Dec 12, 2024

LGTM, thank you!!

Copy link
Member

@sasa-tomic sasa-tomic left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks Nikola!

@NikolaMilosa NikolaMilosa added this pull request to the merge queue Dec 18, 2024
Merged via the queue into master with commit c4bc2ba Dec 18, 2024
25 checks passed
@NikolaMilosa NikolaMilosa deleted the nm-syncing-released-wasms branch December 18, 2024 12:20
github-merge-queue bot pushed a commit that referenced this pull request Dec 19, 2024
As per [this
conversation](#3148 (comment))
this PR focuses on moving `testnet/mainnet_revisions.json` to the root
of repository alongisde `mainnet-canisters.json`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants