Skip to content

fix: update already present repositories#949

Merged
benmss merged 2 commits into
stagingfrom
948-update-existing-repositories
Jan 8, 2025
Merged

fix: update already present repositories#949
benmss merged 2 commits into
stagingfrom
948-update-existing-repositories

Conversation

@benmss

@benmss benmss commented Dec 19, 2024

Copy link
Copy Markdown
Contributor

This fixes the issue with previously cloned repositories not updating when newer changes are present at their remote origins, by using git fetch when the target directory of the clone is already present. This comes with a small performance hit for any analysis performed on a previously cloned target.

Closes #948

@benmss benmss self-assigned this Dec 19, 2024
@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Dec 19, 2024
@benmss benmss force-pushed the 948-update-existing-repositories branch from 7e0110d to 90cd182 Compare December 19, 2024 02:14
Comment thread src/macaron/slsa_analyzer/git_url.py Outdated
Signed-off-by: Ben Selwyn-Smith <benselwynsmith@googlemail.com>
@benmss benmss force-pushed the 948-update-existing-repositories branch from 90cd182 to 6aae4ef Compare December 19, 2024 03:20
@benmss benmss marked this pull request as ready for review December 19, 2024 05:15
@benmss benmss requested a review from behnazh-w as a code owner December 19, 2024 05:16
Comment thread src/macaron/slsa_analyzer/git_url.py
Signed-off-by: Ben Selwyn-Smith <benselwynsmith@googlemail.com>
@benmss benmss merged commit d76b111 into staging Jan 8, 2025
@benmss benmss deleted the 948-update-existing-repositories branch April 8, 2025 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants