Shopify/packwerk

[Feature Request] Detect stale todos

pjambet opened this issue · 1 comments

Description

It would be convenient to have a command that allows the detection of stale todos. This command could be run in CI environments to catch stale todos before they're introduced to the main branch

To Reproduce

Any setup where there is a stale violation, where running bundle exec packwerk update-todos would delete said stale todo.

Expected Behaviour

A command that would detect stale todos, such as returning a non-zero exit code.

Screenshots

N/A

Version Information

  • Packwerk: v3.0.0
  • Ruby 3.1.2

Additional Context

Without a git environment, the only solution I explored is checking md5 checksums before running update-todos, and after, which works, but is somewhat cumbersome.

I believe what you're describing is #298. There is a stale file check in check, but it doesn't seem to catch this. We used to have a command, but check should cover this use-case. I'll close this in favor of the bug issue, but let me know if I missed anything. Thanks!