08. Reviewing pull requests

As a reviewer, you need to ensure

• Put under review badge to the pull request
• Check that the code compiles and the existing tests succeed
• Does the example code implement the pattern correctly and follow good coding practices?
• Does the example code have proper tests and enough test coverage?
• Is the example code commented well enough?
• Is the example code following JavaDoc conventions?
• Are the project coding conventions being followed (Checkstyle handles this)?
• Is the class diagram generated correctly?
• Is the index.md implemented correctly so the pattern will show correctly on the web site?
• Is the originator of the design pattern properly credited?

As a general guideline, pull requests with no activity during the last 30 days will be closed.