Take the dead feature flag ticket. Remove the flag and everything behind the off branch.
DevinDevin Cloud session11:20
Removed the flag and the unreachable branch across four files, kept the on path, and ran the suite. The pull request is open and linked to PLT-412.
PLT-412, proposed change
File
Change
State
checkout/flags.ts
Remove the retired flag
Ready
checkout/legacy.tsx
Delete the off branch
Ready
tests/checkout.spec.ts
Drop the off branch cases
Ready
Draftbranch chore/retire-checkout-flag, not merged
Remove the retired checkout flag
The flag has been on in every environment since the spring, so the off branch was unreachable code carrying its own tests. The flag, its branch and its cases are gone and the suite passes.
4 files and 1 pull request
Approved by Ines Duarte
Merged, and PLT-412 closes with the reasoning in this thread.