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

fix: remove unused update-output-files arg from e2e-tests #50

Merged
merged 3 commits into from
Jun 16, 2024

Conversation

kayagokalp
Copy link
Owner

@kayagokalp kayagokalp commented Jun 16, 2024

Removes unused param from e2e-test binary, also update ci to run coverage on PRs once.

@kayagokalp kayagokalp self-assigned this Jun 16, 2024
@kayagokalp kayagokalp added tech-debt Everything related to tech-debt and various chores ci Everything CI related labels Jun 16, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.37%. Comparing base (d62a318) to head (690cb9d).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   28.33%   28.37%   +0.03%     
==========================================
  Files          20       20              
  Lines        1426     1424       -2     
==========================================
  Hits          404      404              
+ Misses       1022     1020       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kayagokalp kayagokalp merged commit 56ec267 into master Jun 16, 2024
8 checks passed
@kayagokalp kayagokalp deleted the kayagokalp/fix-ci branch June 16, 2024 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Everything CI related tech-debt Everything related to tech-debt and various chores
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants