Skip to content

Stop using CODECOV_TOKEN#144

Merged
dfed merged 1 commit intomainfrom
dfed--no-longer-require-codecov-token
Jan 8, 2025
Merged

Stop using CODECOV_TOKEN#144
dfed merged 1 commit intomainfrom
dfed--no-longer-require-codecov-token

Conversation

@dfed
Copy link
Copy Markdown
Owner

@dfed dfed commented Jan 8, 2025

Forked repos should be able to contribute and have CI succeed. Example failure: https://github.com/dfed/SafeDI/actions/runs/12676458517/job/35331202575

I have made the token not required in Codecov

@dfed dfed self-assigned this Jan 8, 2025
@dfed dfed force-pushed the dfed--no-longer-require-codecov-token branch from 4519d31 to 3594d26 Compare January 8, 2025 19:25
@dfed dfed mentioned this pull request Jan 8, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.91%. Comparing base (b6d4ccb) to head (3594d26).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #144      +/-   ##
==========================================
- Coverage   99.91%   99.91%   -0.01%     
==========================================
  Files          32       32              
  Lines        3467     3465       -2     
==========================================
- Hits         3464     3462       -2     
  Misses          3        3              

see 1 file with indirect coverage changes

@dfed dfed marked this pull request as ready for review January 8, 2025 20:36
@dfed dfed merged commit 4919abf into main Jan 8, 2025
@dfed dfed deleted the dfed--no-longer-require-codecov-token branch January 8, 2025 20:36
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.

1 participant