-
Notifications
You must be signed in to change notification settings - Fork 67
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
Require pull request reviews before merging #228
Comments
@BebeSparkelSparkel I am just a one-shot contributor to this package (in Sep 2021), having done some simple maintenance that does not requires to understand the inner workings of this package. So, please do not consider me as reviewer for this package. But thanks for asking! |
@andreasabel Thanks for letting me know! |
@BebeSparkelSparkel are you interested in becoming a maintainer? |
I use this a lot so yes |
@snoyberg can you work out a transition. Do you want to add @BebeSparkelSparkel as a maintainer? |
@gregwebs I already have write permissions. Do I need any others? |
write on Github? Then there's the issue of releasing versions to Hackage |
Good point about hackage. |
@BebeSparkelSparkel if you send me your Hackage username and which packages you'd like to maintain there, happy to add you. |
@andreasabel @gregwebs @kbillings @maxcan @MaxGabriel
Are any of you still interested and have time to review/approve pull requests?
If so, what do you think about implementing required pull request views before merging?
The text was updated successfully, but these errors were encountered: