Hi devs,
We’re Thursday and Thursday is XWiki Day! :)
I’d like to propose that we do our first PR Review Day today. The goal is to review all
the JIRA issues that have patches or PRs in them
(
http://jira.xwiki.org/secure/IssueNavigator.jspa?reset=true&mode=hide&a…)
and:
- apply them if possible
- comment on them if something is missing/not right
- set the proper “Pull Request Status” field value: Awaiting Committer Feedback, Awaiting
Contributor Feedback, None
- ping the contributor if we’ve already asked for some info and we didn’t get it
- if some PR are almost good then spend the extra time to finish them to allow applying
the PR
Process:
- when you start reviewing a PR please assign it to yourself to set the lock on it
- when you’re done, if the issue wasn’t closed, then unassign yourself
Please record:
- how many issues you’ve reviewed
- how many PRs you’ve applied
WDYT?
Let’s do it! :)
Thanks
-Vincent