Skip to content

Conversation

@Kayanski
Copy link
Contributor

@Kayanski Kayanski commented Jun 7, 2024

This PR aims at adding below second block time to cw-orch-daemon to accelerate transaction waiting time in case of local testing for instance

Checklist

  • Changelog updated.
  • Docs updated.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 7, 2024

Deploying cw-orchestrator with  Cloudflare Pages  Cloudflare Pages

Latest commit: be1f6e3
Status: ✅  Deploy successful!
Preview URL: https://1b4a1c4d.cw-orchestrator.pages.dev
Branch Preview URL: https://nicolas-orc-139-below-second.cw-orchestrator.pages.dev

View logs

@codecov
Copy link

codecov bot commented Jun 7, 2024

Codecov Report

Attention: Patch coverage is 35.29412% with 22 lines in your changes missing coverage. Please review.

Project coverage is 56.5%. Comparing base (0a2f299) to head (be1f6e3).
Report is 2 commits behind head on main.

Additional details and impacted files
Files Coverage Δ
cw-orch-daemon/src/queriers/node.rs 44.7% <100.0%> (+0.5%) ⬆️
packages/clone-testing/src/core.rs 84.5% <ø> (ø)
cw-orch-daemon/src/core.rs 58.6% <0.0%> (ø)
cw-orch-daemon/src/tx_broadcaster.rs 75.7% <0.0%> (-0.5%) ⬇️
cw-orch-daemon/src/env.rs 57.3% <11.1%> (-14.4%) ⬇️

@Kayanski Kayanski requested a review from Buckram123 June 11, 2024 09:48
@Kayanski Kayanski merged commit 9ad1948 into main Jun 11, 2024
@Kayanski Kayanski deleted the nicolas/orc-139-below-second-block-time branch June 11, 2024 13:29
Comment on lines +24 to 26
- Daemon : Changed return types on daemon queriers to match CosmWasm std types
- Daemon: Added below second block time.
- Cw-orch : Separate osmosis test tube from cw-orch. Its not available in its own crate `cw-orch-osmosis-test-tube`
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we keep France punctuation for cw-orch changelog 🤣 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hahahhaha, maybe

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