-
Notifications
You must be signed in to change notification settings - Fork 673
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(build-and-test): add cmake-build-type #1965
Conversation
Codecov ReportBase: 11.13% // Head: 10.27% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1965 +/- ##
==========================================
- Coverage 11.13% 10.27% -0.87%
==========================================
Files 1203 1158 -45
Lines 86271 82148 -4123
Branches 20832 19115 -1717
==========================================
- Hits 9610 8440 -1170
+ Misses 66497 64565 -1932
+ Partials 10164 9143 -1021
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There isn't enough storage in GitHub-hosted runners. |
b389f8e
to
efb237c
Compare
efb237c
to
518aa34
Compare
Testing again (split the workflows): |
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
553adcc
to
52b9e88
Compare
This pull request has been automatically marked as stale because it has not had recent activity. |
Description
Add the debug build to the scheduled test in order to test in more detail.
Related:
signal_processing
tests rely on uninitialized variable #1959Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.