Remove the unchanged gate chain left without a map #1225
Labels
No labels
P0
P1
P2
P3
bug
category/code-change
category/cross-repo
category/eval-run
category/investigation
category/no-test-target
category/owner-ruling
category/prose-artifact
category/recurring-chore
category/structure
category/umbrella
crc-refactor
create
delete
enhancement
filed-by/agent
filed-by/user
frozen
lint-rule
needs-info
needs-triage
next
plugin/cc-architect
plugin/os
plugin/os-adr
plugin/os-aidd-lint
plugin/os-backlog
plugin/os-context
plugin/os-doc-hygiene
plugin/os-sdlc
plugin/os-vault
project/cc-os
ready-for-agent
ready-for-human
recurring
review
rule-proposal
update
waiting
wayfinder:grilling
wayfinder:map
wayfinder:map
wayfinder:research
wayfinder:task
wayfinder:task
wontfix
worklist/deviations
worklist/lint-rule
worklist/new-implement-build
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
jared/cc-os#1225
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
Blocked by: #1224.
After ticket 1 deletes the
structure-changemap, no map uses theunchangedgate.The gate chain stays as dead code: 8 lib and bin files and 6 tests.
Jared has not ruled on the parent delete choice; it was predicted on his behalf (option A, 75% confidence) and is open for his review, so this ticket depends on that review.
Reproduce
n/a: this is dead code after ticket 1; no failing behavior exists.
Approach
Delete the
unchangedgate chain and every reference to it.The chain is
Gate::Name::UNCHANGED,Gate::Commands#unchanged_command,Gate::UnchangedCheckwithbase_checkout,comparison, andmeasurement,Implementation::Acceptance#unchanged_check, theunchanged-checkentry inBoot::Commands, andbin/os-sdlc-unchanged-check.Delete the support file for the retired test as well.
Required work:
unchangedinplugins/os-sdlc/lib/os_sdlc/gate/name.rbunchanged_commandinplugins/os-sdlc/lib/os_sdlc/gate/commands.rbUnchangedCheckinplugins/os-sdlc/lib/**/*.rbunchanged_checkinplugins/os-sdlc/lib/os_sdlc/implementation/acceptance.rbunchanged-checkinplugins/os-sdlc/lib/os_sdlc/boot/commands.rbunchangedinplugins/os-sdlc/bin/*UnchangedCheckScenarioinplugins/os-sdlc/tests/support/*.rbOrchestrator work: run
bin/refresh-pluginsafter the deletion.Rejected: keep the chain for future maps — no map names it and no ticket plans one; git history keeps the code.
Acceptance criteria
unchangedgate exists.bindirectory, when listed, then noos-sdlc-unchanged-checkexecutable exists.unchanged-checkentry exists.Owner sketch
Retired tests
plugins/os-sdlc/tests/boot_unchanged_check_test.rb
Origin
VERDICT: CREATE