Skip to content
This repository was archived by the owner on Jul 23, 2020. It is now read-only.

Content reviewers FTW #24

Open
chartgerink opened this issue Dec 8, 2017 · 4 comments
Open

Content reviewers FTW #24

chartgerink opened this issue Dec 8, 2017 · 4 comments

Comments

@chartgerink
Copy link
Member

chartgerink commented Dec 8, 2017

This issue maintains the list of people who have expressed interest in reviewing. If you are interested in reviewing and want to be pinged on the Pull Requests drop a line here! 🥂 Expressing interest is not a commitment, and you can review how much you like when you'd like. 👍

@rainsworth
@jibe-b
@emaassen

For a short intro to reviewing a pull request (97 seconds!) view this short clip:

Watch the video

If there are comments about the reviewing process, feel free to drop them here. Also feel free to drop me a line here if you do not want to be pinged anymore.

@jibe-b
Copy link

jibe-b commented Jan 9, 2018

Thank you for leading a dynamics and opening the door for lazy cooperation through reviews!

@jibe-b
Copy link

jibe-b commented Jan 10, 2018

@chartgerink How much do you like my review? May I ask you to review my review with review quality criteria in mind? ;)

@chartgerink
Copy link
Member Author

I appreciate the directness with which you formulate your review and you have some good pointers @jibe-b 💯

I might need to add a bit on scope of the project, because I notice some of your comments and other's want to go deeper, which I understand, and I am trying to balance between detail and engaging onboarding. Maybe sections on Further reading would be worthwhile after the exercises, for those interested?

@jibe-b
Copy link

jibe-b commented Feb 9, 2018

I approve your idea :) a Further reading section makes sense!

@chartgerink chartgerink mentioned this issue Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants