logoalt Hacker News

andaiyesterday at 3:09 PM2 repliesview on HN

Interesting, from the title I thought it was intentional, as a "forced code review." Apparently not, but now I really like that idea!


Replies

jakub_gyesterday at 10:53 PM

I always wanted to make feature flags system where each FF must declare an expiration date max 1 year in the future and start failing CI beyond that date to force someone to reevaluate and clean up.

It's just too easy to keep adding new feature flags and never removing them. Until one day the FF backend goes down and you have 300 FFs all evaluate to false.

show 1 reply
adrianpikeyesterday at 3:21 PM

We've done that at a few places I've been at - it's tricky because if the failure is too short its just annoying toil, but if it's too long there's risk of losing context and having to remember what the heck we were thinking.

Overall it's still net positive for me in certain cases of enforcing things to be temporary, or at least revisited.

show 1 reply