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

Add dummy Debug instance to authority discovery service. #9156

Merged
3 commits merged into from
Jun 22, 2021

Conversation

eskimor
Copy link
Member

@eskimor eskimor commented Jun 21, 2021

Useful when added to other structs which are supposed to have a Debug instance. This would save me a newtype wrapper in Polkadot.

@eskimor eskimor 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 C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jun 21, 2021
More idiomatic print

Co-authored-by: Pierre Krieger <pierre.krieger1708@gmail.com>
@eskimor
Copy link
Member Author

eskimor commented Jun 22, 2021

bot merge

@ghost
Copy link

ghost commented Jun 22, 2021

Waiting for commit status.

@ghost ghost merged commit 0982f10 into master Jun 22, 2021
@ghost ghost deleted the rk-authority-debug-instance branch June 22, 2021 13:24
athei pushed a commit that referenced this pull request Jun 25, 2021
* Add dummy Debug instance to AuthorityDiscoveryService.

* Update client/authority-discovery/src/service.rs

More idiomatic print

Co-authored-by: Pierre Krieger <pierre.krieger1708@gmail.com>

Co-authored-by: Pierre Krieger <pierre.krieger1708@gmail.com>
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants