Update coverity scan build and upload to single step

FreeRTOS / FreeRTOS-Kernel

FreeRTOS kernel files only, submoduled into https://github.com/FreeRTOS/FreeRTOS and various other repos.

https://www.FreeRTOS.org

MIT License

2.76k stars 1.12k forks source link

Update coverity scan build and upload to single step #902

Closed tony-josi-aws closed 11 months ago

tony-josi-aws commented 11 months ago

Description

This PR updates the coverity scan yml script to do the coverity scan build and upload to single step.

Test Steps

Tested on fork

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

sonarcloud[bot] commented 11 months ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

Skptak commented 11 months ago

Either this or #903 should fix this issue I introduced this bug when splitting the creation of the tar file and the uploading of it into separate steps. I had split these into separate steps so that if future changes were made to use the created coverity report that the creation and upload of it were seperated.

The tar file needs to be moved out of the build directory if used in a following step. Or the following step needed to move into the build directory, as is done in the first directory.

I approved this PR as this will also fix the current problem with this scan failing.