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

Add Commit Message Table and Add Commit Messages to Facade Collection #2879

Merged
merged 6 commits into from
Aug 5, 2024

Conversation

IsaacMilarky
Copy link
Contributor

Description

  • Previously Facade did not care about the commit message
  • Now, we have a new table that has a unique record for each repo and commit hash with the hash's corresponding commit message
  • We bulk insert the commit message data

Notes for Reviewers
Should be tested on a range of repositories before being put into production. For now this is a draft PR until we can be more confidant in it.

Signed commits

  • Yes, I signed my commits.

@IsaacMilarky IsaacMilarky added the add-feature Adds new features label Jul 25, 2024
augur/tasks/git/facade_tasks.py Outdated Show resolved Hide resolved
augur/tasks/git/facade_tasks.py Outdated Show resolved Hide resolved
augur/tasks/git/facade_tasks.py Outdated Show resolved Hide resolved
augur/tasks/git/facade_tasks.py Outdated Show resolved Hide resolved
Signed-off-by: Isaac Milarsky <[email protected]>
Signed-off-by: Isaac Milarsky <[email protected]>
@IsaacMilarky IsaacMilarky marked this pull request as ready for review August 5, 2024 22:38
Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@sgoggins sgoggins merged commit 0abf235 into dev Aug 5, 2024
9 checks passed
@IsaacMilarky IsaacMilarky deleted the add-commit-message-data branch August 5, 2024 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add-feature Adds new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants