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

Decision Process Implementation POC #1

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

Conversation

joulei
Copy link
Owner

@joulei joulei commented Oct 26, 2022

This a working draft for the decision process implementation.
Our intent is to get a working POC with merge and hold to get a first round of feedback.

Co-authored by @mcass19

Mauricio Cassola and others added 12 commits November 11, 2022 18:11
DB support for issue decision state
* add initial parsing for merge and hold commands

* cleanup format checking in parser
* Automate documentation updates for rust-lang/rust

* docs-update: Switch to using a separate repo for creating the commits.

* Fix recent_commits pagination.

* Automatically re-scan GitHub commits every 30 minutes

This lets us recover from faults in the GitHub API more quickly.

* Do not expect sha field in API response

We should eventually add in-CI testing for this kind of oops (it's
fairly common when we change these) or move to auto-generated GitHub API
code -- but for now this fixes commit fetching.

Co-authored-by: Eric Huss <[email protected]>
Co-authored-by: Mark Rousskov <[email protected]>
This reverts commit 6fd8baa.
* add comment builder for the decision state

* improve error handling

* add factory for user status and add tests

* cleanup unnecessary code
* Parse team in command

* Use parsed team in handler
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant