[CI] New workflows order part1 #791
Conversation
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
lukaszstolarczuk
force-pushed
the
order-ci-pt1
branch
from
3bbdcfe to
6864590
Compare
lukaszstolarczuk
force-pushed
the
order-ci-pt1
branch
2 times, most recently
from
7178d9d to
17f8762
Compare
it's already covered in "Basic builds".
It seems Scorecard is accepting reusable workflows now. Verified on my fork where workflows works properly and there's no change in the Score.
lukaszstolarczuk
force-pushed
the
order-ci-pt1
branch
from
17f8762 to
fde1ca7
Compare
Latest distros do not allow global pip installation. Python packages have to be installed in venv, on Linux. Signed-off-by: Lukasz Dorau <[email protected]>
lukaszstolarczuk
force-pushed
the
order-ci-pt1
branch
from
fde1ca7 to
10b9391
Compare
|
Code coverage did not change - still 80,1% - sounds like I didn't break anything with this change :) |
| @@ -1,3 +1,4 @@ | |||
| # Run tests with valgrind intstrumentation tools: memcheck, drd, helgrind | |||
There was a problem hiding this comment.
intstrumentation -> instrumentation
There was a problem hiding this comment.
I'll fix that in the next PR; I'd like to omit waiting for the whole CI to re-run 😄
3 participants
Please review each commit separately. Verified on my fork it all works as expected.
Notes:
.github/workflowsdirectory.Checklist