Skip to content

Conversation

@Kayanski
Copy link
Contributor

@Kayanski Kayanski commented Dec 4, 2023

This PR aims at returning results for failing behavior instead of panicking.
This is used in https://github.com/AbstractSDK/cw-orch-interchain/pull/19

@codecov
Copy link

codecov bot commented Dec 4, 2023

Codecov Report

Merging #280 (b931fb4) into main (40a19ac) will decrease coverage by 0.4%.
Report is 3 commits behind head on main.
The diff coverage is 91.3%.

❗ Current head b931fb4 differs from pull request most recent head cfd9626. Consider uploading reports for the commit cfd9626 to get more accurate results

Additional details and impacted files
Files Coverage Δ
cw-orch-daemon/src/json_file.rs 100.0% <100.0%> (ø)
cw-orch-daemon/src/state.rs 80.8% <90.0%> (+0.8%) ⬆️

... and 6 files with indirect coverage changes

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 4, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: cfd9626
Status:⚡️  Build in progress...

View logs

@Kayanski Kayanski requested a review from CyberHoward December 8, 2023 11:25
@Kayanski Kayanski merged commit 36abd7d into main Dec 12, 2023
@Kayanski Kayanski deleted the fix/no-unwrap-in-daemon-state branch December 12, 2023 13:59
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