Alex/kaido-core
SHA256
Watch
1
0
Fork
You've already forked kaido-core
 0
Code Issues 11 Pull requests 1 Releases Activity Actions

Remove process references from code #76

New issue
Open
opened 2026-11-06 21:26:44 +01:00 by Alex · 0 comments
Alex commented 2026-11-06 21:26:44 +01:00
Owner

Problem

  • Code comments, test names, test file names and SQL refer to ADR numbers, work packages, waves, reviews and proof-of-concept phases.
  • These references age and mean nothing to a reader of the code.

Change

  • Remove the references; comments state only what the code does and why.
  • Rename *_review_test.go files and test helpers named after a review or a work package.
  • Product terms stay: commit review, wave mode in deploy-config.sh.

Validation

  • go vet on the changed Go packages.
## Problem - Code comments, test names, test file names and SQL refer to ADR numbers, work packages, waves, reviews and proof-of-concept phases. - These references age and mean nothing to a reader of the code. ## Change - Remove the references; comments state only what the code does and why. - Rename `*_review_test.go` files and test helpers named after a review or a work package. - Product terms stay: commit review, wave mode in `deploy-config.sh`. ## Validation - `go vet` on the changed Go packages.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
134-reply-to-before-routing
119-example-firstboot
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Alex/kaido-core#76
No description provided.