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

Carthage: Staging parameters #4364

Merged
merged 5 commits into from
Oct 13, 2022

Conversation

mnaamani
Copy link
Member

@mnaamani mnaamani commented Oct 11, 2022

Updated runtime parameters as per #4355 to have a more easily testable staging network with similar constraints to production.

Updated feature names to match rust convention (removed underscore so testing_runtime -> testing-runtime)

┆Issue is synchronized with this Asana task by Unito

@vercel
Copy link

vercel bot commented Oct 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
pioneer-testnet ⬜️ Ignored (Inspect) Oct 13, 2022 at 6:50AM (UTC)

Copy link
Contributor

@Lezek123 Lezek123 left a comment

Choose a reason for hiding this comment

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

LGTM, just a few typos

@@ -183,7 +183,7 @@ mod tests {
}

// This test does not make sense for `testing_runtime`, because of 1s blocks
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// This test does not make sense for `testing_runtime`, because of 1s blocks
// This test does not make sense for `testing-runtime`, because of 1s blocks

@@ -211,7 +211,7 @@ mod tests {
}

// This test does not make sense for `testing_runtime`, because of 1s blocks
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// This test does not make sense for `testing_runtime`, because of 1s blocks
// This test does not make sense for `testing-runtime`, because of 1s blocks

@@ -0,0 +1,332 @@
//! This module contains staging parameters for the runtime codex proposals.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
//! This module contains staging parameters for the runtime codex proposals.
//! This module contains playground parameters for the runtime codex proposals.

@@ -1,72 +1,84 @@
//! This module contains staging parameters for the runtime codex proposals.
//! This module contains default "production" parameters configuration for the runtime codex proposals.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
//! This module contains default "production" parameters configuration for the runtime codex proposals.
//! This module contains staging parameters for the runtime codex proposals.

@mnaamani mnaamani merged commit 69447d7 into Joystream:carthage Oct 13, 2022
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.

None yet

2 participants