Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

test(driver): fix workflow errors #245

Merged
merged 1 commit into from
May 27, 2023
Merged

test(driver): fix workflow errors #245

merged 1 commit into from
May 27, 2023

Conversation

davidtaikocha
Copy link
Member

@davidtaikocha davidtaikocha commented May 27, 2023

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Overall, this Pull Request looks good. However, there are a few minor issues that should be addressed.

  • There is a typo in the repeating variable TREASURY - it is used twice but with different values. This should be reviewed and corrected.
  • The commit message could be more descriptive and provide more information about the changes being made.

@davidtaikocha davidtaikocha requested a review from RogerLamTd May 27, 2023 01:53
@davidtaikocha davidtaikocha enabled auto-merge (squash) May 27, 2023 01:53
@codecov
Copy link

codecov bot commented May 27, 2023

Codecov Report

Merging #245 (c12cd58) into main (8033e31) will decrease coverage by 0.51%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
- Coverage   54.39%   53.88%   -0.51%     
==========================================
  Files          36       36              
  Lines        3550     3550              
==========================================
- Hits         1931     1913      -18     
- Misses       1365     1382      +17     
- Partials      254      255       +1     

see 2 files with indirect coverage changes

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

@davidtaikocha davidtaikocha merged commit fe86c0e into main May 27, 2023
@davidtaikocha davidtaikocha deleted the fix-action-errors branch May 27, 2023 02:02
vhjiang pushed a commit to taikoverse/taiko-degen-client that referenced this pull request Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants