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

Implement the Runtime API subsystem #1494

Merged
9 commits merged into from
Jul 30, 2020
Merged

Implement the Runtime API subsystem #1494

9 commits merged into from
Jul 30, 2020

Conversation

rphmeier
Copy link
Contributor

Closes #1419

I'd also like a test that this propagates errors to the sender, but the type system more or less ensures that and the mock_impl_runtime_apis! macro doesn't let us return errors at the moment.

@rphmeier rphmeier 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 Jul 28, 2020
@rphmeier rphmeier requested a review from ordian July 30, 2020 01:21
Copy link
Member

@ordian ordian left a comment

Choose a reason for hiding this comment

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

Looks good, just a few unused deps

@rphmeier
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Jul 30, 2020

Waiting for commit status.

@ghost
Copy link

ghost commented Jul 30, 2020

Checks failed; merge aborted.

@rphmeier
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Jul 30, 2020

Waiting for commit status.

@ghost ghost merged commit f772196 into master Jul 30, 2020
@ghost ghost deleted the rh-runtime-api-subsystem branch July 30, 2020 18:15
ordian added a commit that referenced this pull request Jul 31, 2020
* master:
  Candidate Validation Subsystem (#1432)
  reduce slash defer durations (#1504)
  Implement the Runtime API subsystem (#1494)
  Companion for #6610 (Balances Weight Trait) (#1425)
  [CI] Publish draft release redux (#1493)
  Update README docs related to local build (#1479)
  Add a default trie-memory-tracker feature to the cli (#1502)
  Companion PR for `Add a `DefaultQueue` type alias to remove the need to use `sp_api::TransactionFor`` (#1499)
  Fix bitfield signing (#1466)
  Update Substrate, bump versions, clean up sort (#1496)
  Bump Substrate
This pull request was closed.
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Runtime API Subsystem
2 participants