-
Notifications
You must be signed in to change notification settings - Fork 731
feat: pick repo implementation [CM-2384] #3254
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
feat: pick repo implementation [CM-2384] #3254
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR titles must follow Conventional Commits. Love from, Your reviewers ❤️.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great 👍 added one nitpick
-- Processing state and priority | ||
state VARCHAR(50) NOT NULL DEFAULT 'pending', | ||
priority INTEGER NOT NULL DEFAULT 0, -- 0=urgent, 1=high, 2=normal | ||
priority INTEGER NOT NULL DEFAULT 1, -- 0=urgent, 1=high, 2=normal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: would be nice if we can add documentation on this as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I've added it to the TODOs 🫡
d57f7f8
into
feat/git_integration_processing_worker
Changes proposed ✍️
Implement a pick repository from the database with locking by priority to prevent multiple workers from processing the same repo at the same time.
What
copilot:summary
copilot:poem
Why
How
copilot:walkthrough
Checklist ✅
Feature
,Improvement
, orBug
.