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

chore(sns): Remove obsolete scripts for Swap canister upgrades #4238

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

aterga
Copy link
Member

@aterga aterga commented Mar 5, 2025

This PR removes the obsolete scripts from testnet/tools/nns-tools/ that were previously used to upgrade Swap via NNS proposals. This workflow is obsolete due to Swap now being a proper SNS canister, which means it is now upgraded by the SNSs themselves.

@aterga aterga marked this pull request as ready for review March 5, 2025 08:10
@aterga aterga requested a review from a team as a code owner March 5, 2025 08:10
@github-actions github-actions bot added the chore label Mar 5, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

If this pull request affects the behavior of any canister owned by
the Governance team, remember to update the corresponding
unreleased_changes.md file(s).

To acknowldge this reminder (and unblock the PR), dismiss this
code review by going to the bottom of the pull request page, and
supply one of the following reasons:

  1. Done.

  2. No canister behavior changes.

@aterga aterga dismissed github-actions[bot]’s stale review March 5, 2025 08:11

No canister changes

@aterga aterga enabled auto-merge March 5, 2025 08:11
@aterga aterga added this pull request to the merge queue Mar 5, 2025
Merged via the queue into master with commit e2a9827 Mar 5, 2025
21 checks passed
@aterga aterga deleted the arshavir/NNS1-3419 branch March 5, 2025 16:10
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.

2 participants