Skip to content

fix: execute evmos build only once per PR - #353

Merged
jbowen93 merged 2 commits into
mainfrom
tzdybal/evmos-once
Apr 11, 2022
Merged

fix: execute evmos build only once per PR#353
jbowen93 merged 2 commits into
mainfrom
tzdybal/evmos-once

Conversation

@tzdybal

@tzdybal tzdybal commented Apr 10, 2022

Copy link
Copy Markdown
Contributor

Resolves #347.

@tzdybal
tzdybal requested a review from jbowen93 April 10, 2022 17:57
@tzdybal tzdybal self-assigned this Apr 10, 2022
@tzdybal tzdybal added the C:ci Component: Continuous Integration label Apr 10, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #353 (47f9e4e) into main (e917abf) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #353      +/-   ##
==========================================
- Coverage   59.13%   59.09%   -0.05%     
==========================================
  Files          44       44              
  Lines        7183     7183              
==========================================
- Hits         4248     4245       -3     
- Misses       2364     2366       +2     
- Partials      571      572       +1     
Impacted Files Coverage Δ
rpc/json/service.go 63.80% <0.00%> (-1.85%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e917abf...47f9e4e. Read the comment docs.

Comment thread .github/workflows/build-evmos.yml Outdated
Co-authored-by: John Adler <adlerjohn@users.noreply.github.com>
@jbowen93
jbowen93 merged commit a57e85f into main Apr 11, 2022
@jbowen93
jbowen93 deleted the tzdybal/evmos-once branch April 11, 2022 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:ci Component: Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't run build evmos twice in CI

4 participants