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

Remove deprecated API #4993

Merged
merged 5 commits into from
Feb 20, 2020
Merged

Conversation

stanislav-tkach
Copy link
Contributor

Remove deprecated api and dead code before 2.0 release.

Closes #4935.

@stanislav-tkach stanislav-tkach added the A3-in_progress Pull request is in progress. No review needed at this stage. label Feb 19, 2020
)?
{
// Run compatibility fallback for v3.
self.api.map_api_result(|api| {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure if this should be removed.

@stanislav-tkach
Copy link
Contributor Author

#[allow(dead_code)] still remains in macros and tests.

@stanislav-tkach stanislav-tkach changed the title [In Progress] Remove deprecated API Remove deprecated API Feb 19, 2020
@stanislav-tkach stanislav-tkach added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Feb 19, 2020
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

Besides the added dead_code (should be removed) it looks good.

frame/babe/src/mock.rs Outdated Show resolved Hide resolved
@gnunicorn gnunicorn added A5-grumble and removed A0-please_review Pull request needs code review. labels Feb 20, 2020
@shawntabrizi shawntabrizi merged commit dbeebe2 into master Feb 20, 2020
@shawntabrizi shawntabrizi deleted the stas-remove-deprecated-and-dead-4935 branch February 20, 2020 16:03
General-Beck pushed a commit to General-Beck/substrate that referenced this pull request Mar 4, 2020
* Remove deprecated API

* Remove (some) allow(dead_code)

* Bump spec_version

* Fix import, remove allow dead code.

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove all deprecated and allow(deprecated) APIs
5 participants