Here are some of our tips for conducting Code Reviews the SweetOps way. If you haven’t already, become familiar with our Best Practices and Terraform Best Practices. Use the “Suggest” feature as much as possible. This makes it quick and easy for the contributor to accept or dismiss the recommendations. Use proper markdown in suggestions (e.g. code blocks) Always be polite and appreciative of the contributions! Use emoticons to up-vote other comments (rather than +1 comments) Use ChatOps commands like /rebuild-readme or /terraform-fmt to fix common problems Use ChatOps commands like /test all, /test bats, /test readme, /test terratest to run integration tests Recommend changes to better conform to our best-practices.
We’ve written thousands of lines of Makefile. These are our best practices from the trenches.
Our opinionated best-practices for Terraform