feat: Added csdiff and git packages#292
Conversation
dirgim
requested review from
14rcole,
Josh-Everett,
dheerajodha,
dirgim,
hongweiliu17,
jencull,
jsztuka,
kasemAlem and
ralphbean
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #292 +/- ##
========================================
Coverage ? 100.00%
========================================
Files ? 18
Lines ? 498
Branches ? 0
========================================
Hits ? 498
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. |
jperezdealgaba
force-pushed
the
csdiff
branch
3 times, most recently
from
123a053 to
f67dbe6
Compare
|
@ralphbean Would you mind giving a review on this? |
|
@ralphbean @14rcole @Josh-Everett Could any of you review it and approve it/comment it? Thank you! |
|
/ok-to-test |
|
@ralphbean Would you mind enabling the last CI test? I am not able to trigger it/merge this |
|
@jperezdealgaba if you rebase your commit on the latest |
jperezdealgaba
force-pushed
the
csdiff
branch
2 times, most recently
from
3c4a2d3 to
fd8d6b2
Compare
csdiff and git packages are installed in order to be used by SAST tasks Package will be used for parsing results and generating fingerprints and results files Signed-off-by: Juan Pérez de Algaba <jperezde@redhat.com>
jperezdealgaba
force-pushed
the
csdiff
branch
from
fd8d6b2 to
a7a9551
Compare
jperezdealgaba
changed the title
|
@dirgim I rebased the PR and GitHub is still showing that I need the approval from one maintainer for the workflow:
P.S.: I added the installation of the git packages to this PR as it will be also needed. I hope it is not a problem |
|
All checks passed! I have no merge rights in this repo |
5 participants
csdiff and git packages are installed in order to be used by SAST tasks to parse the results and generate fingerprinting.
This will be used by future SAST tasks provided by the OpenScanHub team, for example: https://issues.redhat.com/browse/OSH-737