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

Revision exception votes for PRs #18

Closed
wants to merge 8 commits into from

Conversation

kathrynmohror
Copy link
Collaborator

This introduces the concept of a Revision Exception vote, which will allow forward progress on PRs even if minor modifications are made after the quarterly meeting review period deadline or after first vote. If all eligible voting members of the ASC vote 'yes' on the Revision Exception vote, then the PR may proceed for normal voting including the modifications. If any member votes 'no' then the proposer must either re-read the PR with the modifications or proceed to normal vote without the modifications.

@jjhursey jjhursey added the Eligible Eligible for consideration by ASC label Sep 4, 2020
@jjhursey
Copy link
Member

jjhursey commented Sep 4, 2020

Please use emoji reactions ON THIS COMMENT to indicate your position on this proposal.

  • You do not need to vote on every proposal
  • If you have no opinion, don't vote - that is also useful data
  • If you've already commented on this issue, please still vote so
    we know your current thoughts
  • Not all proposals solve exactly the same problem, so we may end
    up accepting proposals that appear to have some overlap
    This is not a binding majority-rule vote, but it will be a very
    significant input into the corresponding ASC decision.

Here are the meanings for the emojis:

  • Hooray or Rocket: I support this so strongly that I
    want to be an advocate for it
  • Heart: I think this is an ideal solution
  • Thumbs up: I'd be happy with this solution
  • Confused: I'd rather we not do this, but I can tolerate it
  • Thumbs down: I'd be actively unhappy, and may even consider
    other technologies instead
    If you want to explain in more detail, feel free to add another
    comment, but please also vote on this comment.

@jjhursey
Copy link
Member

jjhursey commented Oct 1, 2020

4Q 2020 ASC Meeting - Oct. 1, 2020

  • Passed first vote: 10 yes / 0 no / 0 abstain

@jjhursey jjhursey added First Vote Passed ASC first vote passed Proposed Ready for review labels Oct 1, 2020
@jjhursey jjhursey added the Accepted ASC second vote passed. Accepted! label Feb 18, 2021
@jjhursey
Copy link
Member

1Q 2021 ASC Meeting (Feb. 18, 2021)

  • Passed second Vote: 9 yes / 0 no / 0 abstain / 5 absent

@jjhursey
Copy link
Member

Due to some problems with git on the branch a fresh version of this PR was created in #24

@jjhursey jjhursey closed this Feb 26, 2021
@kathrynmohror kathrynmohror deleted the revision-exception branch February 27, 2021 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted ASC second vote passed. Accepted! Eligible Eligible for consideration by ASC First Vote Passed ASC first vote passed Proposed Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants