|
|
|
|
|
by mkal_tsr
4410 days ago
|
|
Horrible implementation approach. Mixing runtime exceptions with project management that you're not guaranteed to hit while testing? lol. A pre-commit hook is the way to go, * Post-commit: You'll always catch the exception, but you'll be able to commit expired TODOs * Pre-commit: you'll have to explicitly change the expired TODOs either by removing them (irrelevant TODO, todo became todone through a refactor, etc.), or discussing a better end-date ... but either way, you're forced to address the expired TODO and not run the risk of committing expired TODOs nor not taking a test path that would miss the expired TODO exception |
|