feat(code-review): add resource links to all 6 topics (#10012)
* feat(vibe-coding): add resource links to all 54 topics Added verified article links to all vibe-coding roadmap topics. Every topic now has at least 3 high-quality resource links. - 25 topics filled from 0 links - 7 topics filled from 1 link to 3 - 7 topics filled from 2 links to 3 - 15 existing topics already had 3+ links (preserved) - 96 new links added, all verified HTTP 200 - Zero duplicate URLs across new additions - No body text modified, no existing links removed * feat(code-review): add resource links to all 6 topics - index: 1 link (Google Engineering Practices) - code-style: 3 links (Google, Graphite, Refactoring Guru) - documentation: 3 links (Write the Docs, Heretto, Google) - tests: 3 links (IBM, Microsoft, Martin Fowler) - implementation-semantics: 3 links (IBM, Redwerk, Codacy) - api-semantics: 3 links (Fern, RESTful API, Microsoft Azure) 16 total links, 0 duplicates, all verified HTTP 200 --------- Co-authored-by: nehaturov-hue <nehaturov-hue@users.noreply.github.com>
This commit is contained in:
1 parent
2ae46be018
commit
d909c0cba7
6 files changed
+35
-1
No files matched your search
@@ -6,3 +6,9 @@
|
||||
- Clean split of API/internals without internals leaking into the API?
|
||||
- Are there no breaking changes to user-facing parts (API classes, configuration, metrics, log formats, etc)?
|
||||
- Is a new API generally useful and not overly specific to a single use case?
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@article@API Design Best Practices Guide - Fern](https://buildwithfern.com/post/api-design-best-practices-guide)
|
||||
- [@article@RESTful API Resource Naming](https://restfulapi.net/resource-naming/)
|
||||
- [@article@Web API Design Best Practices - Microsoft Azure](https://learn.microsoft.com/en-us/azure/architecture/best-practices/api-design)
|
||||
@@ -4,3 +4,9 @@
|
||||
- Does it adhere to the agreed-upon naming conventions?
|
||||
- Is it DRY?
|
||||
- Is the code sufficiently "readable" (method lengths, etc.)?
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@article@The Standard of Code Review - Google](https://google.github.io/eng-practices/review/reviewer/standard.html)
|
||||
- [@article@Creating a Coding Style Guide for Your Team - Graphite](https://graphite.com/guides/creating-coding-style-guide)
|
||||
- [@article@Code Smells - Refactoring Guru](https://refactoring.guru/refactoring/smells)
|
||||
@@ -3,3 +3,9 @@
|
||||
- Are the new features reasonably documented?
|
||||
- Are all relevant types of documentation covered, such as README, API docs, user guide, reference docs, etc?
|
||||
- Is the documentation understandable and free of significant typos and grammar mistakes?
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@article@How to Write Software Documentation - Write the Docs](https://www.writethedocs.org/guide/writing/beginners-guide-to-docs/)
|
||||
- [@article@8 Essential Code Documentation Best Practices - Heretto](https://www.heretto.com/blog/best-practices-for-writing-code-documentation)
|
||||
- [@article@Looking for Things in a Code Review - Google](https://google.github.io/eng-practices/review/reviewer/looking-for.html)
|
||||
@@ -8,3 +8,9 @@
|
||||
- Is it secure (i.e., no SQL injections, etc.)?
|
||||
- Is it observable (i.e., metrics, logging, tracing, etc.)?
|
||||
- Do newly added dependencies pull their weight? Is their license acceptable?
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@article@Three Pillars of Observability: Logs, Metrics and Traces - IBM](https://www.ibm.com/think/insights/observability-pillars)
|
||||
- [@article@Security Code Review Checklist - Redwerk](https://redwerk.com/blog/security-code-review-checklist/)
|
||||
- [@article@What Is Clean Code? - Codacy](https://blog.codacy.com/what-is-clean-code)
|
||||
@@ -1 +1,5 @@
|
||||
#
|
||||
#
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@official@Google Engineering Practices](https://github.com/google/eng-practices)
|
||||
@@ -5,3 +5,9 @@
|
||||
- Are corner cases tested?
|
||||
- Is it using unit tests where possible, integration tests where necessary?
|
||||
- Are there tests for NFRs, e.g. performance?
|
||||
|
||||
Learn more from the following resources:
|
||||
|
||||
- [@article@Unit Testing Best Practices - IBM](https://www.ibm.com/think/insights/unit-testing-best-practices)
|
||||
- [@article@Integration Testing - Microsoft Engineering Playbook](https://microsoft.github.io/code-with-engineering-playbook/automated-testing/integration-testing/)
|
||||
- [@article@The Practical Test Pyramid - Martin Fowler](https://martinfowler.com/articles/practical-test-pyramid.html)
|
||||
Reference in new issue
Block a user