-
Notifications
You must be signed in to change notification settings - Fork 661
Roles and tasks (Component reviews)
Thomas Gohard edited this page Jun 20, 2013
·
4 revisions
Return to component reviews index
- Looks for new pull requests.
- Analyses the code changes, for each new pull request, to see if the code changes affect the structure, presentation or behaviour of one or more existing component or implement one or more new components.
- Creates an issue requesting a component review for each component for which the code changes affect the structure, presentation or behaviour and for each new component.
- Looks for new issues requesting a component review.
- Reviews the component for each component for which a component review has been requested.
- Posts a comment on the issue that requested the component review in which they provide a link to their component review.
- Looks through all open issues requesting a component review for completed component reviews.
- Compiles the individual reviews into a single component review for each component for which there are at least three individual component reviews completed.
- Posts a comment on the issue for which they compiled the individual component reviews in which they provide a link to the compiled component review and closes the issue.
- Creates an issue for each usability recommendation identified in the compile component review.
- Looks through all open issues making a usability recommendation for questions from developers.
- Answers the questions.
- Looks through all open issues making usability recommendations for references to pull requests implementing the usability recommendation.
- Reviews that the implementation of the usability recommendation is done correctly and:
- If the implementation is done correctly, indicates that the implementation is OK from a usability perspective.
- If the implementation is not done correctly, indicates where the implementation fails and works with the developer to remedy the problem.