SDK Update and Multi-Runtime Configuration #476
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* SDK update v1.12.0 * multi-runtime initial configuration * multi-runtime assitional configuration * Deprecate NativeElseWasmExecutor * Test updates * Authority - check for MinAuthorities before removal * Cargo workspace and multi-runtime Braid configuration * Braid configuration updates * Loom Intial configiration and Braid Updates * Loom runtime updates - fee configuration * Loom runtime updates - node and tests
* update CI - missed files
indent error fix
* minor fix - dockerfile * multi-arch docker build fixes * multi-arch docker build profile fix * add missing dependencies to node * distroless docker images * distroless docker images - docker path fixes * distroless docker images - dockerfile and workflow updates * updated Cargo.lock * multi-arch docker build - add missing development headers * multi-arch docker build - add missing / to command * multi-arch docker build - mapi binary correctly * multi-arch docker build - switch to cc-debian12 * multi-arch docker build - move binary to usr/bin * multi-arch docker build - add missing libraries * multi-arch docker build - add missing path * multi-arch build documentation and branch checks * multi-arch removing on-chain-release-build feature tag * doc updates
vatsa287
reviewed
Jun 18, 2024
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
amarts
approved these changes
Jun 23, 2024
| - "*" | ||
| branches: | ||
| - develop | ||
| - release-v[0-9]+.[0-9]+.[0-9]+* |
| BlockLength::max_with_normal_ratio(5 * 1024 * 1024, NORMAL_DISPATCH_RATIO); | ||
| pub const SS58Prefix: u8 = 29; | ||
| pub const Version: RuntimeVersion = VERSION; | ||
| pub const SS58Prefix: u16 = 3893; |
Member
There was a problem hiding this comment.
This number is random, or any particular reason?
| /// Default for Weave | ||
| Weave = 29, | ||
| /// Default for unknown chains | ||
| Default = 42, |
Member
There was a problem hiding this comment.
Should we keep it different from that of substrate's default?
| #[derive(Debug, Clone, Copy)] | ||
| pub enum Ss58AddressFormatPrefix { | ||
| /// Default for Braid | ||
| Braid = 3893, |
Member
There was a problem hiding this comment.
Ok, I see that the previous value is from here.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.