Conversation
There was a problem hiding this comment.
This review was generated by an AI code review assistant and may contain inaccuracies.
Adds a require-copyright-header ESLint rule to enforce Cloudflare copyright headers on all TS/JS files, and backfills 80+ files that were missing them.
- [LOW] The rule lacks auto-fix support, which would make it trivial to remediate violations on new files (see inline suggestion).
|
I'm Bonk, and I've done a quick review of your PR. This PR adds a The rule implementation is correct -- the regex properly matches the expected header format (including year ranges), the base config integration is clean, and the One low-severity suggestion posted: adding |
16162e9 to
ffb16d7
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6268 +/- ##
==========================================
- Coverage 70.69% 70.68% -0.02%
==========================================
Files 417 417
Lines 111111 111111
Branches 18331 18331
==========================================
- Hits 78552 78534 -18
- Misses 21614 21630 +16
- Partials 10945 10947 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I keep forgetting adding copyright headers.