forked from Vulnogram/Vulnogram
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Split REVIEW state #57
Labels
enhancement
New feature or request
Comments
this change needs to be done in a larger context of changes to the state workflow. what states are needed, when and who moves between them, acl's on those changes, notifications on those changes (etc) |
replaces: #42 |
replaces: #41 |
(I moved the 'overall' changes to an 'umbrella' ticket at #61 and renamed this one back to 'Split REVIEW state') |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Split REVIEW state: Since states are our internal state and not Mitres state and can be anything, we could always have a PMC-REVIEW vs SECURITY-REVIEW to make it clear who has the action of doing the review. This is because some PMC's like to have the REVIEW state as part of their process and that makes sense.
This way we could also enforce the state changes (i.e. can't leave security-review without someone in security doing the state change)
The text was updated successfully, but these errors were encountered: