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

Changed update CC thresholds to 51% #6321

Closed
wants to merge 1 commit into from
Closed

Conversation

kevinhammond
Copy link

Pre-submit checklist:

  • Branch
    • [N/A] Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • [N/A] Changelog fragments have been written (if appropriate)
    • [N/A] Relevant tickets are mentioned in commit messages
    • [N/A] Formatting, PNG optimization, etc. are updated
  • PR
    • (For external contributions) Corresponding issue exists and is linked in the description
    • Targeting master unless this is a cherry-pick backport
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

This PR updates the thresholds and comments for the "Update constitutional committee action" in defaultConstitution.json to reflect a late guardrail change from 65% to 51%.

The original high-level guardrail was:

"Update Constitutional committee action thresholds must be in the range 65%-90%"
The new guardrail is:

"Update Constitutional committee action thresholds must be in the range 51%-90%"

Four guardrails are affected: for DReps or for SPOs in both Confidence and No Confidence states

@kevinhammond kevinhammond requested a review from bezirg July 22, 2024 15:48
@bezirg bezirg added the No Changelog Required Add this to skip the Changelog Check label Jul 22, 2024
@bezirg
Copy link
Contributor

bezirg commented Jul 22, 2024

Closed in favor of #6320

@bezirg bezirg closed this Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Required Add this to skip the Changelog Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants