docs(rules): leave commented-out code to the definition of done
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
1 parent
bd4d94e451
commit
0af6b13a16
3 files changed
+1
-3
No files matched your search
@@ -37,7 +37,6 @@
|
|||||||
* Avoid vague names such as `data`, `item`, `value` or `temp` when better names exist.
|
* Avoid vague names such as `data`, `item`, `value` or `temp` when better names exist.
|
||||||
* Remove dead code.
|
* Remove dead code.
|
||||||
* Remove unused imports and dependencies.
|
* Remove unused imports and dependencies.
|
||||||
* Remove commented-out code instead of keeping it in source files.
|
|
||||||
* Add comments only for non-obvious decisions or public contracts.
|
* Add comments only for non-obvious decisions or public contracts.
|
||||||
* Treat Angular, TypeScript and lint warnings seriously.
|
* Treat Angular, TypeScript and lint warnings seriously.
|
||||||
* Do not suppress diagnostics without a concrete reason.
|
* Do not suppress diagnostics without a concrete reason.
|
||||||
|
|||||||
@@ -40,7 +40,7 @@
|
|||||||
* Avoid unnecessary wrapper classes and indirection.
|
* Avoid unnecessary wrapper classes and indirection.
|
||||||
* Prefer composition over inheritance.
|
* Prefer composition over inheritance.
|
||||||
* Remove dead code when it is clearly unused.
|
* Remove dead code when it is clearly unused.
|
||||||
* Remove obsolete comments and commented-out code.
|
* Remove obsolete comments.
|
||||||
* Keep methods and classes focused.
|
* Keep methods and classes focused.
|
||||||
|
|
||||||
## Architecture
|
## Architecture
|
||||||
|
|||||||
@@ -43,7 +43,6 @@
|
|||||||
* Prefer concrete types where no abstraction is required.
|
* Prefer concrete types where no abstraction is required.
|
||||||
* Use descriptive names.
|
* Use descriptive names.
|
||||||
* Remove dead code.
|
* Remove dead code.
|
||||||
* Remove commented-out code.
|
|
||||||
* Add comments only for non-obvious decisions or public contracts.
|
* Add comments only for non-obvious decisions or public contracts.
|
||||||
* Treat compiler, analyzer and binding warnings seriously.
|
* Treat compiler, analyzer and binding warnings seriously.
|
||||||
* Do not suppress warnings without a concrete reason.
|
* Do not suppress warnings without a concrete reason.
|
||||||
|
|||||||
Reference in new issue
Block a user