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.

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

chore(bridge): add a FAILED message status #356

Merged
merged 1 commit into from
Nov 29, 2022
Merged

Conversation

dantaik
Copy link
Contributor

@dantaik dantaik commented Nov 29, 2022

See issue #355

@codecov
Copy link

codecov bot commented Nov 29, 2022

Codecov Report

Merging #356 (7b63737) into main (6500234) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #356   +/-   ##
=======================================
  Coverage   63.43%   63.43%           
=======================================
  Files          88       88           
  Lines        2489     2489           
  Branches      283      283           
=======================================
  Hits         1579     1579           
  Misses        838      838           
  Partials       72       72           
Flag Coverage Δ *Carryforward flag
protocol 58.42% <100.00%> (ø)
relayer 68.92% <ø> (ø) Carriedforward from 6500234
ui 100.00% <ø> (ø) Carriedforward from 6500234

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
...s/protocol/contracts/bridge/libs/LibBridgeData.sol 100.00% <ø> (ø)
packages/protocol/contracts/L1/TaikoL1.sol 22.22% <100.00%> (ø)
.../protocol/contracts/bridge/libs/LibBridgeRetry.sol 88.88% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dantaik dantaik merged commit e97132c into main Nov 29, 2022
@dantaik dantaik deleted the bridge_failed_status branch November 29, 2022 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants