Skip to content

feat: add checks to determine if repo and commit came from provenance #704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 15, 2024

Conversation

benmss
Copy link
Member

@benmss benmss commented Apr 15, 2024

This PR adds a new check that succeeds if the repository URL and commit of the analysis target match those that can be extracted from the provenance. If the repository or provenance do not exist, or do not contain the needed information, or are not identical, this check will fail.

Closes #677

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Apr 15, 2024
@benmss benmss marked this pull request as ready for review April 16, 2024 08:46
@benmss benmss requested review from behnazh-w and tromai as code owners April 16, 2024 08:46
@benmss benmss marked this pull request as draft April 19, 2024 00:32
@behnazh-w
Copy link
Member

@benmss The scope of this PR should be feat.

@benmss benmss changed the title chore: add check to determine if repo and commit came from provenance feat: add check to determine if repo and commit came from provenance Apr 29, 2024
@benmss benmss force-pushed the 677-check-repo-commit-from-provenance branch from ceb45be to a7b2a2a Compare April 30, 2024 23:01
@benmss benmss marked this pull request as ready for review May 1, 2024 01:15
@benmss benmss marked this pull request as draft May 1, 2024 23:50
@benmss benmss changed the title feat: add check to determine if repo and commit came from provenance feat: add checks to determine if repo and commit came from provenance May 7, 2024
@benmss benmss force-pushed the 677-check-repo-commit-from-provenance branch 2 times, most recently from cdec8f2 to dbfd995 Compare May 7, 2024 05:54
@benmss benmss marked this pull request as ready for review May 7, 2024 05:59
@behnazh-w behnazh-w requested a review from nicallen May 8, 2024 00:07
@benmss benmss force-pushed the 677-check-repo-commit-from-provenance branch from e5a567a to af35243 Compare May 13, 2024 02:10
@benmss benmss force-pushed the 677-check-repo-commit-from-provenance branch from af35243 to 1279beb Compare May 13, 2024 02:17
Copy link
Member

@tromai tromai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change. LGTM.

@behnazh-w
Copy link
Member

@benmss Please update the list of checks in docs/source/index.rst at Current checks in Macaron and run make docs-api.

@benmss benmss merged commit 9c44445 into staging May 15, 2024
@tromai tromai deleted the 677-check-repo-commit-from-provenance branch May 15, 2024 23:58
art1f1c3R pushed a commit that referenced this pull request Nov 29, 2024
…#704)

This PR adds two new checks that succeed if the repository URL or commit of the analysis target match those that can be extracted from the provenance, respectively. If the repository or provenance do not exist, or do not contain the needed information, or are not identical, these checks will fail.

Signed-off-by: Ben Selwyn-Smith <[email protected]>
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.

4 participants