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

Add recusal list strategy [recusal-list] #602

Merged
merged 2 commits into from
May 18, 2022

Conversation

bshyong
Copy link
Contributor

@bshyong bshyong commented May 16, 2022

Fixes # . n/a

Changes proposed in this pull request:

  • Adds Recusal List strategy. This is strategy for disallowing certain address from voting due to conflict of interest or other reasons for recusal.

Copy link
Member

@ChaituVR ChaituVR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can rename this to the blacklist strategy? any thoughts?

src/strategies/recusal-list/README.md Outdated Show resolved Hide resolved
@ChaituVR ChaituVR changed the title Add recusal list strategy Add recusal list strategy [recusal-list] May 18, 2022
@bshyong
Copy link
Contributor Author

bshyong commented May 18, 2022

Maybe we can rename this to the blacklist strategy? any thoughts?

I thought about that language but decided on Recusal because a blacklist has a negative connotation and is less descriptive - Recusal explicitly refers to addresses being restricted from voting with regards in governance.

@ChaituVR ChaituVR merged commit ca03f2d into snapshot-labs:master May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants