Closed Paul-GitGuardian closed 9 months ago
Attention: 9 lines
in your changes are missing coverage. Please review.
Comparison is base (
ea75562
) 92.15% compared to head (cb42e1a
) 92.03%.
Files | Patch % | Lines |
---|---|---|
ggshield/core/git_hooks/ci/previous_commit.py | 10.00% | 9 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Good catch, in fact it doesn't clone other branches. I've changed it to use remote branches instead. Not sure if I can test the CI using this branch on a real repository. At least I've tested the git commands in a Gitlab pipeline, they work.
You can test it, just declare a job that will clone and install ggshield from your latest commit (maybe on another repo).
BTW, the CI is failing here
@GG-HH I've tested on a repository and made the necessary changes. Could you review it one last time please?
When pushing a new branch for the first time, the GGshield IAC and SCA CI cannot find the commit to use as ref. As described here,
CI_COMMIT_BEFORE_SHA
is empty in that case. Here we use git commands to find the starting commit of the new branch.