search

AbstractSDK / cw-orchestrator

All-in-one Rust-based CosmWasm testing, scripting, and deployment tool.
https://orchestrator.abstract.money
GNU General Public License v3.0
76 stars 18 forks source link

Logs custom CosmTxResponse instead of raw TxResponse #363

Closed Kayanski closed 5 months ago

Kayanski commented 5 months ago

This PR aims at avoiding to log un-necessary information when debug-logging. It just logs the parsed CosmTxResponse instead of the raw TxResponse

cloudflare-pages[bot] commented 5 months ago

Deploying cw-orchestrator with  Cloudflare Pages  Cloudflare Pages

Latest commit: fb99399
Status: ✅  Deploy successful!
Preview URL: https://4a1f75f4.cw-orchestrator.pages.dev
Branch Preview URL: https://nicolas-orc-106-remove-binar.cw-orchestrator.pages.dev

View logs

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.7%. Comparing base (ff1e75d) to head (fb99399).

Additional details and impacted files | [Files](https://app.codecov.io/gh/AbstractSDK/cw-orchestrator/pull/363?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AbstractSDK) | Coverage Δ | | |---|---|---| | [cw-orch-daemon/src/queriers/node.rs](https://app.codecov.io/gh/AbstractSDK/cw-orchestrator/pull/363?src=pr&el=tree&filepath=cw-orch-daemon%2Fsrc%2Fqueriers%2Fnode.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=AbstractSDK#diff-Y3ctb3JjaC1kYWVtb24vc3JjL3F1ZXJpZXJzL25vZGUucnM=) | `44.6% <100.0%> (ø)` | |
CyberHoward commented 5 months ago

Did you confirm that this stopped the bin blob to be logged on upload?

Kayanski commented 5 months ago

Did you confirm that this stopped the bin blob to be logged on upload?

Yes !