Skip to content
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

refactor!: allow the branch name in the schema of a repository to be null #532

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

behnazh-w
Copy link
Member

This PR changes the Repository schema in the data model to allow a branch name to be None. This is in preparation for mapping artifacts to commits because sometimes we might not be able to find a branch. However, we should still be able to checkout using the resolved commit hash.

Note that the local macaron.db needs to be removed to apply this change in the data model, hence the breaking change scope in the PR title.

@behnazh-w behnazh-w added repository_finder The issues related to the repository finder database The issues related to the database labels Oct 27, 2023
@behnazh-w behnazh-w requested a review from benmss October 27, 2023 01:07
@behnazh-w behnazh-w requested a review from tromai as a code owner October 27, 2023 01:07
@behnazh-w behnazh-w self-assigned this Oct 27, 2023
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Oct 27, 2023
@behnazh-w behnazh-w force-pushed the update-type-branch-repo branch from 3394851 to 75f8ced Compare October 27, 2023 01:13
tromai
tromai previously approved these changes Oct 30, 2023
src/macaron/slsa_analyzer/analyzer.py Show resolved Hide resolved
@tromai tromai dismissed their stale review October 30, 2023 03:23

I mistakenly click the approve button. It was not supposed to be an approval.

@behnazh-w behnazh-w merged commit 1b7740b into staging Oct 31, 2023
12 checks passed
@behnazh-w behnazh-w deleted the update-type-branch-repo branch November 2, 2023 23:30
art1f1c3R pushed a commit that referenced this pull request Nov 29, 2024
…null (#532)

This PR changes the Repository schema in the data model to allow a branch name to be None. This is in preparation for mapping artifacts to commits because sometimes we might not be able to find a branch. However, we should still be able to checkout using the resolved commit hash.

Note that the local `macaron.db` needs to be removed to apply this change in the data model, hence the breaking change scope in the PR title.

Signed-off-by: behnazh-w <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database The issues related to the database OCA Verified All contributors have signed the Oracle Contributor Agreement. repository_finder The issues related to the repository finder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants