Skip to content

chore(nns): add support in release scripts for node-rewards canister#4252

Merged
max-dfinity merged 33 commits intomasterfrom
msum/nrc-canister-A
Mar 28, 2025
Merged

chore(nns): add support in release scripts for node-rewards canister#4252
max-dfinity merged 33 commits intomasterfrom
msum/nrc-canister-A

Conversation

@max-dfinity
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the chore label Mar 5, 2025
@max-dfinity max-dfinity changed the base branch from master to msum/nrc-canister March 5, 2025 23:10
Base automatically changed from msum/nrc-canister to master March 6, 2025 21:23
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch from 490b104 to 5bba0c2 Compare March 20, 2025 23:20
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch 2 times, most recently from 9b84947 to 6570af0 Compare March 24, 2025 18:39
@max-dfinity max-dfinity marked this pull request as ready for review March 24, 2025 18:44
@max-dfinity max-dfinity requested review from a team as code owners March 24, 2025 18:44
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.

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.

@max-dfinity max-dfinity enabled auto-merge March 24, 2025 19:59
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch 3 times, most recently from e5b17ab to 3b5b2f2 Compare March 24, 2025 22:21
@max-dfinity max-dfinity disabled auto-merge March 24, 2025 23:06
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch from 9448f28 to 43ad36e Compare March 24, 2025 23:19
@max-dfinity max-dfinity enabled auto-merge March 25, 2025 21:27
@max-dfinity max-dfinity added this pull request to the merge queue Mar 25, 2025
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch from e8bb927 to b5c266b Compare March 28, 2025 17:38
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch from 0357053 to df90c6f Compare March 28, 2025 19:39
@max-dfinity max-dfinity force-pushed the msum/nrc-canister-A branch from df90c6f to 62d87b4 Compare March 28, 2025 19:41
@max-dfinity max-dfinity added this pull request to the merge queue Mar 28, 2025
Merged via the queue into master with commit e325bb0 Mar 28, 2025
21 checks passed
@max-dfinity max-dfinity deleted the msum/nrc-canister-A branch March 28, 2025 21:37
&GENESIS_TOKEN_CANISTER_ID,
&IDENTITY_CANISTER_ID,
&NNS_UI_CANISTER_ID,
&ICP_LEDGER_ARCHIVE_CANISTER_ID,
Copy link
Contributor

Choose a reason for hiding this comment

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

it's a bit confusing to me to have zero based indexing but then not call this ICP_LEDGER_ARCHIVE_0_CANISTER_ID. isn't that inconsistent?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 @dre @idx @nns-team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants