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

Comments

collation-generation: guide and tidying#1753

Merged
4 commits merged intomasterfrom
ser-collation-fixes
Sep 25, 2020
Merged

collation-generation: guide and tidying#1753
4 commits merged intomasterfrom
ser-collation-fixes

Conversation

@pepyakin
Copy link
Contributor

See each commit.

i noticed these things during my work on #1679

Seems like that was missed during the transition from notions of global and local validation data to persistent and transient
this allows us to avoid piling Error and Result types from different contexts, specifically std vs. module-local
@pepyakin pepyakin added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Sep 25, 2020
@pepyakin pepyakin added this to the Grosvenor milestone Sep 25, 2020
@pepyakin pepyakin requested a review from ordian September 25, 2020 14:45
@ordian ordian added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Sep 25, 2020
@pepyakin
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Sep 25, 2020

Trying merge.

@ghost ghost merged commit 507b9cb into master Sep 25, 2020
@ghost ghost deleted the ser-collation-fixes branch September 25, 2020 16:33
ordian added a commit that referenced this pull request Sep 28, 2020
* master:
  Collator protocol followup (#1741)
  Revert "Remove Old Service, 2nd try (#1732)" (#1758)
  Remove Old Service, 2nd try (#1732)
  collation-generation: guide and tidying (#1753)
  Companion for #7111 (Introduce `cancel_proposal` and `blacklist`) (#1728)
  Parachains: Introduce a dummy module to include the Origin. (#1749)
  provisioner tests: remove tokio from dev-dependencies (#1745)
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants