-
Notifications
You must be signed in to change notification settings - Fork 683
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
Move doxygen to GitHub Actions #1458
Conversation
seladb
commented
Jun 23, 2024
•
edited
Loading
edited
- Move doxygen to GitHub Actions and remove it from Cirrus CI
- Temporary comment-out all Cirrus CI tasks so we can merge PRs
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1458 +/- ##
==========================================
- Coverage 82.66% 82.65% -0.02%
==========================================
Files 171 171
Lines 22585 22585
Branches 8559 8559
==========================================
- Hits 18671 18667 -4
- Misses 3116 3137 +21
+ Partials 798 781 -17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@seladb should It be a different workflow? |
I'm not sure it's necessary, because a major part is to test doxygen which is part of |
@seladb @tigercosmos to me the other reason is that the triggering mode could differs from the build_package but maybe i'm wrong, So it don't have a strong opinion. |