points by fg137 3 days ago | 0 comments
A developer on my team introduced new features behind feature flags despite concerns about whether we should have those features and how they are going to be maintained. Changes touched 200+ files which is half of the files.
Turns out nobody wants to use them, they caused many subtle bugs in existing features and is a distraction for many things we do.
The developer left the team, and I am left with purging all of this.
[–] RiverStone 2 days ago
[–] fg137 22 hours ago
I fully agree that there is a code review problem.
But the management chain doesn't particularly care about that, even less so with AI.
[–] program_whiz 3 days ago
Now imagine if he had 100x the speed! I'm sure AI will help him thoughtfully pay off tech debt at his next company!
200+ files should have triggered a conversation by the team, request to split it up, etc.
If nobody wants to use the features then that work never should have been prioritized or planned. The team should have discussed why it was being worked on if not a priority, etc.
Lots of layers of improvement here beside AI