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 CI code for comment.yml #404

Merged
merged 13 commits into from
Jun 23, 2024
Merged

Fix CI code for comment.yml #404

merged 13 commits into from
Jun 23, 2024

Conversation

hanbings
Copy link
Collaborator

link: #360 and proceed: #400

Copy link

codecov bot commented Jun 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.48%. Comparing base (d329702) to head (49fa96b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   60.48%   60.48%           
=======================================
  Files          32       32           
  Lines        4069     4069           
  Branches      895      895           
=======================================
  Hits         2461     2461           
  Misses       1254     1254           
  Partials      354      354           

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

@hanbings
Copy link
Collaborator Author

I've tested this code with PRs in my fork. There should be no problem.
and... I seem to have forgotten to do a commit squash to clean up the extra commit records. Could you clean them up when you merge them? Thanks! :)

@sylvestre sylvestre merged commit 5b19ea2 into uutils:main Jun 23, 2024
19 checks passed
@sylvestre
Copy link
Contributor

#389 (comment)
seems that it works, bravo :)

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.

2 participants