-
-
Notifications
You must be signed in to change notification settings - Fork 839
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
Update contributing docs #1011
Update contributing docs #1011
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if I can approve but LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome 🥇
@jcugat Would you be keen to join @encode/maintainers? No expectations, but you'd be able to approve PRs and move them to a mergeable state — extra eyes and reducing the bus factor is always great. :-) What do others think? |
I'm still making my way through all the code, but sure, I'll try to give a hand where I can 🙂 |
Hmm, doesn't look like I can add members to the team. @tomchristie Any chance you can help out? |
Co-authored-by: Josep Cugat <jcugat@gmail.com>
Co-authored-by: Josep Cugat <jcugat@gmail.com>
Co-authored-by: Josep Cugat <jcugat@gmail.com>
6664a87
to
2c755ca
Compare
Closes #936