search

softwareunderground / 52things

52 Things You Should Know About Geocomputing
102 stars 61 forks source link

Reviewing_submissions.md #9

Open rowanc1 opened 7 years ago

rowanc1 commented 7 years ago

I was interested to see that you didn't promote pull request reviews as an initial chance to jump into the reviewing process.

@lheagy and I have found that pretty interesting in the geosci sphere when getting a bunch of authors to collaborate on textbook chapters. Might be something worth exploring over here as well?

Example: https://github.com/geoscixyz/em/pull/480

kwinkunks commented 7 years ago

My feeling was that it might lead some people into a sticky merge. I'm definitely open to giving this a try, if you'd like to pick on one of the more Git-savvy authors to experiment on :)

I guess I wish it was possible to accept and reject individual commits in a PR. (If this is possible, I haven't figured out how to do it.)

mtb-za commented 7 years ago

Feel free to review mine as PRs, if you prefer. :)