Skip to content
New issue

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

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

Already on GitHub? # to your account

chore!: remove alias exports DefaultContext and EyreContext #181

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LeoniePhiline
Copy link
Contributor

DefaultContext and EyreContext exist as alias exports enabled by the anyhow feature.

However, they are unused within eyre; neither are handlers of such names exposed by anyhow.

This changeset removes both above mentioned handler aliases.

Closes #135.

BREAKING CHANGE: The alias exports are undocumented but nevertheless part of the crate's public API.

`DefaultContext` and `EyreContext` exist as
alias exports enabled by the `anyhow` feature.

However, they are unused within `eyre`; neither
are handlers of such names exposed by `anyhow`.

This changeset removes both handler aliases.

Closes [eyre-rs#135][135].

[135]: eyre-rs#135

BREAKING CHANGE: The alias exports are undocumented
                 but nevertheless part of the
                 crate's public API.
@LeoniePhiline LeoniePhiline force-pushed the chore/135-remove-unused-alias-exports branch from 2dbd5b1 to e461b8c Compare July 11, 2024 11:54
@LeoniePhiline
Copy link
Contributor Author

Rebased.

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

Successfully merging this pull request may close these issues.

[breaking] remove alias exports DefaultContext and EyreContext
1 participant