daily-email: add 2023-08-29

CI pipelines are an automated code review

src/content/daily-email/2023-08-29.md

@@ -0,0 +1,34 @@
+---
+title: >
+  CI pipelines are an automated code review
+pubDate: 2023-08-29
+permalink: >
+  archive/2023/08/29/ci-pipelines-are-an-automated-code-review
+tags:
+  - code-review
+  - ci-pipelines
+---
+
+I've worked on various teams over the last 13 years on which we've needed to do feature branches, pull requests and code reviews.
+
+If the request isn't approved by (usually) two people, it won't be merged.
+
+Instead of focusing on the problem that needed to be solved and how I'd done it, many reviews focused on the small details.
+
+Do the lines have the correct number of spaces before them?
+
+Do the comments end with a full stop?
+
+Do the lines wrap at the correct point, and are your variable names in the right case?
+
+Essentially, does the code comply with the agreed coding standards?
+
+## Here's the thing
+
+Whilst important (you want the code to follow standards and be in a consistent format), doing these checks manually is not a good use of time and is not what the code review should focus on.
+
+These checks can be automated using CI pipelines or Git hooks to run tools like PHPCS to review and sometimes fix coding standards issues.
+
+Automating these checks means the Developers can focus on what they should be reviewing.
+
+How are they solving the problem, not how many spaces is the code indented by.