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

Companion for #7363 (Account-providers ref-counting) #2152

Merged
merged 6 commits into from
Jan 16, 2021
Merged

Companion for #7363 (Account-providers ref-counting) #2152

merged 6 commits into from
Jan 16, 2021

Conversation

gavofyork
Copy link
Member

@gavofyork gavofyork commented Dec 21, 2020 •
edited by shawntabrizi
Loading

@gavofyork gavofyork added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes A9-companion-backlog A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. C1-low PR touches the given topic and has a low impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. labels Dec 21, 2020
@gavofyork gavofyork mentioned this pull request Dec 21, 2020
Merged
@gavofyork gavofyork closed this Jan 16, 2021
@gavofyork gavofyork reopened this Jan 16, 2021
@gavofyork gavofyork merged commit 400ecc5 into master Jan 16, 2021
@gavofyork gavofyork deleted the gav-account-provider-refs branch January 16, 2021 19:18
xlc pushed a commit to xlc/polkadot that referenced this pull request Jan 26, 2021
@gavofyork @shawntabrizi @xlc
Companion for paritytech#7363 (Account-providers ref-counting) (parit…
bb1cd7f 
…ytech#2152)

* Fix

* Fix

* Bump Substrate

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
@pablojorge pablojorge mentioned this pull request Mar 9, 2021
Merged
@gui1117 gui1117 added D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited. and removed A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. labels Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gui1117 gui1117 gui1117 approved these changes

@kianenigma kianenigma kianenigma approved these changes

@shawntabrizi shawntabrizi shawntabrizi approved these changes

Assignees
No one assigned
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@gavofyork @shawntabrizi @kianenigma @gui1117 @paritytech-ci