Skip to content

examples: Use anyhow#107

Merged
zeenix merged 1 commit intovarlink:masterfrom
cgwalters:drop-chainerror
Nov 27, 2024
Merged

examples: Use anyhow#107
zeenix merged 1 commit intovarlink:masterfrom
cgwalters:drop-chainerror

Conversation

@cgwalters
Copy link
Collaborator

This is a very widely used error crate suitable for apps and examples exactly like these, in contrast to chainerror
which is only used by this project. Prep for reworking our error handling more generally.

@coveralls
Copy link

coveralls commented Nov 27, 2024

Pull Request Test Coverage Report for Build 12050663215

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.111%

Totals Coverage Status
Change from base Build 12050584654: 0.0%
Covered Lines: 2803
Relevant Lines: 4908

💛 - Coveralls

Copy link
Contributor

@zeenix zeenix left a comment

Choose a reason for hiding this comment

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

For sure! LGTM. :)

@zeenix zeenix enabled auto-merge November 27, 2024 12:43
@zeenix
Copy link
Contributor

zeenix commented Nov 27, 2024

@cgwalters I believe I fixed the CI in #108. Please rebase once that's merged. 🙏

This is a very widely used error crate suitable for apps and examples
exactly like these, in contrast to `chainerror`
which is only used by this project. Prep for reworking our
error handling more generally.

Signed-off-by: Colin Walters <walters@verbum.org>
@zeenix zeenix merged commit 8a7ee17 into varlink:master Nov 27, 2024
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.

3 participants