JohnCosta27/JohnTech
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

update: Coding Review Process.md

Browse Source
This commit is contained in:
John Costa
2026-04-03 21:01:36 +01:00
parent 19baf1946c
commit 11b9f68b02
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
content/blog/Coding Review Process.md
View File

@@ -20,7 +20,7 @@ This should be done in a separate environment to the one you code in. For me, th
1. Does each commit _still_ make sense? 1. Does each commit _still_ make sense?
2. __High level overview__: Does it actually work? Are you wiring the correct functions to the correct place? We often miss the bigger picture when we're down in the code. 2. __High level overview__: Does it actually work? Are you wiring the correct functions to the correct place? We often miss the bigger picture when we're down in the code.
3. Does the architecture make sense? Are the functions you choose and the abstraction you made worth it? 3. Does the architecture make sense? Are the functions you choose and the abstraction you made worth it?
4. Does it work E2E? 4. Does it work end-to-end?
5. __Nits__. Do the names make sense, is the spacing correct. Less important, but still very important. If you can't understand your own code, how can others? 5. __Nits__. Do the names make sense, is the spacing correct. Less important, but still very important. If you can't understand your own code, how can others?
## Optional: Have AI review your code ## Optional: Have AI review your code