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

ci: Do not run code coverage on forks #1900

Merged

Conversation

marcospereira
Copy link
Contributor

What?

Disable code coverage uploading when running on forks. This is useful so that
contributors can enable GitHub Actions on their forks without having false
negatives due to lack of codecov configuration.

Which rule and warnings did you add?

None.

Actions checklist

  • Implemented Rule, added Warnings
  • Added tests on checks
  • Added tests on fixers
  • Updated diktat-analysis.yml
  • Updated available-rules.md

Copy link
Contributor Author

@marcospereira marcospereira left a comment

Choose a reason for hiding this comment

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

Thanks for reviewing, @akuleshov7. I replied with a few questions.

.github/workflows/build_and_test.yml Show resolved Hide resolved
.github/workflows/build_and_test.yml Show resolved Hide resolved
@orchestr7
Copy link
Member

Thanks for reviewing, @akuleshov7. I replied with a few questions.

lgtm, I am trying to understand why CI has freezes (today GitHub had outage)

@orchestr7 orchestr7 closed this Jan 9, 2024
@orchestr7 orchestr7 reopened this Jan 9, 2024
@orchestr7
Copy link
Member

Reopened PR to restart CI actions

Copy link

codecov bot commented Jan 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (435e499) 78.27% compared to head (66e231f) 78.26%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1900      +/-   ##
============================================
- Coverage     78.27%   78.26%   -0.02%     
  Complexity     2478     2478              
============================================
  Files           134      134              
  Lines          8724     8724              
  Branches       2216     2216              
============================================
- Hits           6829     6828       -1     
- Misses          906      907       +1     
  Partials        989      989              

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

@marcospereira
Copy link
Contributor Author

Thank you, @akuleshov7. It seems this is good to go now. :-)

@orchestr7 orchestr7 merged commit 65d7ba9 into saveourtool:master Jan 10, 2024
20 of 25 checks passed
@orchestr7
Copy link
Member

@marcospereira merged, thank you for contribution!

@marcospereira marcospereira deleted the ci/disable-jobs-when-forks branch January 10, 2024 21:45
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.

None yet

2 participants