Improve accuracy and usefulness of the "No TODOs" CI workflow

rdw-archive / WebClient

[Proof of Concept] Browser-based client capable of rendering networked multiplayer games using modern web technologies

1 stars 0 forks source link

Closed Duckwhale closed 2 years ago

Duckwhale commented 2 years ago

There were some issues that I stumbled upon while wrapping up #130 , which should now be resolved:

The workflow no longer fails if changes to the TODO checking script itself are made (silly)
It also ignores removed lines, and only indicates failure when newly-added "temporary" comments are detected
Optional improvement: It now displays more context to make troubleshooting the errors easier