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.

Sign up for GitHub

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

Jump to bottom

Add must_use to Error constructors #229

Merged
merged 5 commits into from Mar 7, 2022
Merged

Add must_use to Error constructors #229

merged 5 commits into from Mar 7, 2022

Commits on Mar 7, 2022

  1. Add ui test for writing anyhow instead of bail 

    Currently fails:

    Expected test case to fail to compile, but it succeeded.
    @dtolnay
    dtolnay committed Mar 7, 2022
    Copy the full SHA
    06826d1 View commit details
    Browse the repository at this point in the history

  2. Add must_use to Error constructors

    @dtolnay
    dtolnay committed Mar 7, 2022
    Copy the full SHA
    1af20cc View commit details
    Browse the repository at this point in the history

  3. Add must_use to value created by anyhow! macro

    @dtolnay
    dtolnay committed Mar 7, 2022
    Copy the full SHA
    ffa919c View commit details
    Browse the repository at this point in the history

  4. Remove must_use call from ensure! and bail! expansions

    @dtolnay
    dtolnay committed Mar 7, 2022
    Copy the full SHA
    573689b View commit details
    Browse the repository at this point in the history

  5. Update must-use test stderr

    @dtolnay
    dtolnay committed Mar 7, 2022
    Copy the full SHA
    cf5d619 View commit details
    Browse the repository at this point in the history