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

Remove CanAuthorWith #10966

Closed
bkchr opened this issue Mar 2, 2022 · 2 comments
Closed

Remove CanAuthorWith #10966

bkchr opened this issue Mar 2, 2022 · 2 comments
Assignees
Labels
I7-refactor Code needs refactoring.

Comments

@bkchr
Copy link
Member

bkchr commented Mar 2, 2022

We have this CanAuthorWIth trait that is being used by the authorship implementations to find out if the node can author with the current on chain runtime. Currently we are using the native runtime version for this. With the removal of the native runtime, we will not have the native runtime version anymore. So, we can not use this one to check if the current node can build with the on chain runtime. As this feature was not really used that much and it was only about a nice way of telling "your node is outdated". If the authoring interface is broken, it would fail anyway to produce new blocks etc.

@bkchr bkchr added the I7-refactor Code needs refactoring. label Mar 2, 2022
@AurevoirXavier
Copy link
Contributor

Just remove all the related code simply?
Would like to know more detail.

@bkchr
Copy link
Member Author

bkchr commented Apr 18, 2022

What details are you interested in?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
I7-refactor Code needs refactoring.
Projects
Status: done
Development

No branches or pull requests

3 participants