-
Notifications
You must be signed in to change notification settings - Fork 51
Add s32k1xx coverage and merge reports #253
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
base: main
Are you sure you want to change the base?
Conversation
040e69c to
5a2550f
Compare
|
Are you planning to fix the merge conflicts? |
53e7de1 to
a3ccd59
Compare
|
I have fixed merge conflicts. |
| # FIXME: It's questionable whether we want to have a dependency to an | ||
| # external service for generating these badges. This introduces a possible | ||
| # cause of instabilities in case the external service becomes unavailable, | ||
| # as already happened in the CI. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shamitha-shashidhara, why was this comment removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johannes-esr I have added it back.
- Integrated tests-s32k1xx-debug into coverage workflow - Merged coverage reports for single badge and HTML output - Relative path to repo root - Excluded all third-party and mock files from coverage Change-Id: Ia1a8c46c4929494f64fa3613f811f38e380bfad2
a3ccd59 to
c5eb27d
Compare
|
@johannes-esr : Can you please check if the result was uploaded correctly to be ready for download? |
originates from CR #136307