Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated Error::description #633

Conversation

AnderEnder
Copy link

Error::description has been documented as soft-deprecated since 1.27.0 (17 months ago). It is going to be hard-deprecated soon.

This PR:

  • Removes all implementations of description in all error types

Related PR: rust-lang/rust#66919

BurntSushi added a commit that referenced this pull request Jan 9, 2020
PR #633 removed these methods, but we can't do that without making a
breaking change release. Removing deprecated methods isn't worth doing a
breaking change release, so we instead simply allow them for now by
squashing the warnings.

Closes #633
@BurntSushi
Copy link
Member

Thanks for this PR! Unfortunately, we can't just remove methods like this that are part of a crate's public API. It would require a semver breaking change release, which I don't think is worth doing just for this. Instead, in #637, I've simply squashed the deprecation warnings. I can remove them on the next semver breaking change release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants