Skip to content

ci: update Unified CI configuration#305

Merged
galargh merged 4 commits intomainfrom
web3-bot/sync
May 17, 2023
Merged

ci: update Unified CI configuration#305
galargh merged 4 commits intomainfrom
web3-bot/sync

Conversation

@galargh
Copy link
Copy Markdown
Contributor

@galargh galargh commented May 17, 2023

No description provided.

@galargh galargh requested a review from a team as a code owner May 17, 2023 07:57
@galargh
Copy link
Copy Markdown
Contributor Author

galargh commented May 17, 2023

I'm going to merge it after verifying all the flags were applied correctly.

Copy link
Copy Markdown
Contributor

@Jorropo Jorropo left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented May 17, 2023

Codecov Report

Merging #305 (e1738e7) into main (fb3abdd) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #305   +/-   ##
=======================================
  Coverage   48.04%   48.05%           
=======================================
  Files         279      279           
  Lines       33497    33497           
=======================================
+ Hits        16095    16098    +3     
+ Misses      15715    15713    -2     
+ Partials     1687     1686    -1     

see 10 files with indirect coverage changes

Copy link
Copy Markdown
Contributor Author

@galargh galargh left a comment

Choose a reason for hiding this comment

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

Forgot to actually update the env 🤦 I'm going to cherry-pick this fix in uCI later.

Comment thread .github/workflows/go-test.yml Outdated
Comment thread .github/workflows/go-test.yml Outdated
@galargh galargh merged commit ff20a4d into main May 17, 2023
@hacdias hacdias deleted the web3-bot/sync branch June 27, 2023 11:01
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