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

Cleanup GitHub Actions #360

Closed
wants to merge 13 commits into from
Closed

Cleanup GitHub Actions #360

wants to merge 13 commits into from

Conversation

Byrdman32
Copy link
Member

No description provided.

@Byrdman32 Byrdman32 added enhancement Requests for new features or improvements to existing features. formatting Issues related to code formatting, style enforcement, or linting tasks. actions Tasks or issues related to GitHub Actions or CI/CD automation workflows. labels Jan 7, 2025
@Byrdman32 Byrdman32 self-assigned this Jan 7, 2025
@Byrdman32 Byrdman32 requested a review from a team as a code owner January 7, 2025 03:08
Copy link

deepsource-io bot commented Jan 7, 2025

Here's the code health analysis summary for commits d96e7f5..120f375. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Shell LogoShell✅ SuccessView Check ↗
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource C & C++ LogoC & C++✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@Byrdman32
Copy link
Member Author

PLEASE DO NOT REVIEW YET. I'M TROUBLESHOOTING THE ACTIONS.

@Byrdman32 Byrdman32 changed the title Consolidate GitHub Actions Cleanup GitHub Actions Jan 7, 2025
@Byrdman32
Copy link
Member Author

I have decided not to merge the Unit and Integration Tests and Code Coverage GitHub Actions because of how much it would end up cluttering up the action environment and how it would increase risk of failures.

Copy link

codecov bot commented Jan 7, 2025

Codecov Report

Attention: Patch coverage is 48.57143% with 18 lines in your changes missing coverage. Please review.

Project coverage is 24.21%. Comparing base (d96e7f5) to head (120f375).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
tests/Unit/src/util/NumberOperations.cc 0.00% 13 Missing ⚠️
src/util/TimeOperations.hpp 70.00% 1 Missing and 2 partials ⚠️
tests/main.cc 80.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           development     #360       +/-   ##
================================================
- Coverage        35.59%   24.21%   -11.39%     
================================================
  Files               80       80               
  Lines             6434     6377       -57     
  Branches          2834     2847       +13     
================================================
- Hits              2290     1544      -746     
- Misses            4144     4584      +440     
- Partials             0      249      +249     
Files with missing lines Coverage Δ
src/AutonomyLogging.cpp 92.63% <100.00%> (-3.49%) ⬇️
tests/main.cc 80.00% <80.00%> (ø)
src/util/TimeOperations.hpp 70.00% <70.00%> (ø)
tests/Unit/src/util/NumberOperations.cc 0.00% <0.00%> (-100.00%) ⬇️

... and 43 files with indirect coverage changes

Components Coverage Δ
algorithms 59.11% <ø> (-21.77%) ⬇️
drivers 23.58% <ø> (-1.03%) ⬇️
handlers 0.00% <ø> (ø)
interfaces 26.46% <ø> (-1.77%) ⬇️
states 0.00% <ø> (ø)
util 32.91% <70.00%> (-11.84%) ⬇️
vision 13.86% <ø> (-3.31%) ⬇️
src 43.47% <100.00%> (-2.65%) ⬇️

@Byrdman32 Byrdman32 added the 1-Star Indicates a very easy task or challenge, suitable for beginners with minimal effort required. label Jan 7, 2025
@Byrdman32 Byrdman32 closed this Jan 7, 2025
@Byrdman32 Byrdman32 deleted the topic/coverage branch January 8, 2025 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-Star Indicates a very easy task or challenge, suitable for beginners with minimal effort required. actions Tasks or issues related to GitHub Actions or CI/CD automation workflows. enhancement Requests for new features or improvements to existing features. formatting Issues related to code formatting, style enforcement, or linting tasks.
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

1 participant