Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Substrate 13843 (Asset Freezing) Companion#2437

Merged
paritytech-processbot[bot] merged 10 commits into
masterfrom
joe-asset-freezecreating
May 8, 2023
Merged

Substrate 13843 (Asset Freezing) Companion#2437
paritytech-processbot[bot] merged 10 commits into
masterfrom
joe-asset-freezecreating

Conversation

@joepetrowski

Copy link
Copy Markdown
Contributor

@joepetrowski joepetrowski added A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. T1-runtime This PR/Issue is related to the topic “runtime”. labels Apr 6, 2023
@paritytech-ci paritytech-ci requested review from a team April 6, 2023 19:09

@NachoPal NachoPal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should consider if it really makes sense to run migrations in testing parachains (penpal and rococo-parachain). As testing parachains they are expected to start with a fresh state. Unless we have a live parachain using one of those runtimes I wouldn't bother adding/removing migrations there.

Comment thread parachains/runtimes/assets/westmint/src/lib.rs Outdated
@paritytech-ci paritytech-ci requested a review from a team April 11, 2023 08:59
@joepetrowski

Copy link
Copy Markdown
Contributor Author

bot rebase

@paritytech-processbot

Copy link
Copy Markdown

Rebased

@muharem

muharem commented May 8, 2023

Copy link
Copy Markdown
Contributor

bot rebase

@paritytech-processbot

Copy link
Copy Markdown

Rebased

@paritytech-processbot

Copy link
Copy Markdown

Waiting for commit status.

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

Labels

A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. T1-runtime This PR/Issue is related to the topic “runtime”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants