Skip to content

refactor(nns): Move governance::init to its own crate to further split type dependencies#490

Merged
max-dfinity merged 11 commits into
masterfrom
detangle-governance-api-types
Jul 24, 2024
Merged

refactor(nns): Move governance::init to its own crate to further split type dependencies#490
max-dfinity merged 11 commits into
masterfrom
detangle-governance-api-types

Conversation

@max-dfinity
Copy link
Copy Markdown
Contributor

@max-dfinity max-dfinity commented Jul 19, 2024

This pulls out a struct into its own crate that is a common reason for other crates to depend the ic_nns_governance crate. This further helps separate out the NNS Governance canister from other parts of the code

@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch 2 times, most recently from 19a47fe to 367a21d Compare July 19, 2024 21:19
@max-dfinity max-dfinity changed the title Detangle governance api types refactor(nns): Move governance::init to its own crate - further dependency detangling Jul 19, 2024
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch 4 times, most recently from b3bd085 to 0397896 Compare July 19, 2024 22:15
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch 3 times, most recently from da0a570 to 37b54ce Compare July 23, 2024 22:52
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch from 58b984a to 608331b Compare July 23, 2024 23:08
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch from 2189752 to 632bcd7 Compare July 23, 2024 23:48
@max-dfinity max-dfinity changed the title refactor(nns): Move governance::init to its own crate - further dependency detangling refactor(nns): Move governance::init to its own crate to further split type dependencies Jul 24, 2024
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch 2 times, most recently from 3aabd3f to 35bde28 Compare July 24, 2024 00:24
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch 2 times, most recently from cf4d362 to e4db739 Compare July 24, 2024 00:44
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch from e4db739 to e46a5d2 Compare July 24, 2024 00:49
@max-dfinity max-dfinity force-pushed the detangle-governance-api-types branch from ae9acb7 to 230ca90 Compare July 24, 2024 04:14
@max-dfinity max-dfinity marked this pull request as ready for review July 24, 2024 14:28
@max-dfinity max-dfinity requested review from a team as code owners July 24, 2024 14:28
Comment thread rs/nns/governance/api/src/pb.rs
Comment thread rs/tests/src/nns_tests/sns_deployment.rs
Comment thread rs/nns/governance/init/BUILD.bazel
@max-dfinity max-dfinity added this pull request to the merge queue Jul 24, 2024
Merged via the queue into master with commit 2926051 Jul 24, 2024
@max-dfinity max-dfinity deleted the detangle-governance-api-types branch July 24, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants