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

Use Foundry Workflow Again #168

Merged
merged 3 commits into from
Sep 5, 2022
Merged

Use Foundry Workflow Again #168

merged 3 commits into from
Sep 5, 2022

Conversation

trajan0x
Copy link
Contributor

@trajan0x trajan0x commented Sep 5, 2022

Description

Foundry nightly release should work now making a6b87ca and #161 no longer neccesary (workarounds for foundry-rs/foundry#3072 and foundry-rs/foundry#3067 respectively

@github-actions github-actions bot added go Pull requests that update Go code M-ci Module: CI labels Sep 5, 2022
@trajan0x trajan0x changed the title Ci/foundry redo Use Foundry Workflow Again Sep 5, 2022
@codecov
Copy link

codecov bot commented Sep 5, 2022

Codecov Report

Merging #168 (468cec4) into master (a6b87ca) will increase coverage by 0.33604%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##              master        #168         +/-   ##
===================================================
+ Coverage   51.81590%   52.15194%   +0.33604%     
===================================================
  Files            141         144          +3     
  Lines           5975        6134        +159     
  Branches          76         104         +28     
===================================================
+ Hits            3096        3199        +103     
- Misses          2581        2633         +52     
- Partials         298         302          +4     
Impacted Files Coverage Δ
...acts/contracts/registry/AbstractNotaryRegistry.sol 71.42857% <0.00000%> (-3.57144%) ⬇️
packages/contracts/test/utils/SynapseTest.sol 0.00000% <0.00000%> (ø)
...ackages/contracts/test/harnesses/OriginHarness.sol 75.00000% <0.00000%> (ø)
...es/contracts/test/harnesses/DestinationHarness.sol 100.00000% <0.00000%> (ø)
packages/contracts/contracts/hubs/ReportHub.sol 100.00000% <0.00000%> (ø)
...racts/contracts/registry/AbstractGuardRegistry.sol 100.00000% <0.00000%> (ø)
...ages/contracts/test/harnesses/ReportHubHarness.sol 100.00000% <0.00000%> (ø)
packages/contracts/contracts/Origin.sol 86.02151% <0.00000%> (+3.09468%) ⬆️
packages/contracts/contracts/Destination.sol 59.16667% <0.00000%> (+5.19842%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@trajan0x trajan0x merged commit 467e555 into master Sep 5, 2022
@trajan0x trajan0x deleted the ci/foundry-redo branch September 5, 2022 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code M-ci Module: CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant