chore: add deprecation warning for eslint v9#3067
Conversation
Greptile SummaryThis PR adds a deprecation warning to
Confidence Score: 4/5Not safe to merge until the placeholder URL is replaced with the actual ESLint v10 migration guide link. The change itself is simple and correct in approach, but the placeholder URL src/cli/format/index.ts — line 19 needs the real migration guide URL. Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant CLI as pepr format (CLI)
participant Log as Log.warn
User->>CLI: runs `pepr format`
CLI->>Log: emit deprecation warning (ESLint v9 → v10)
CLI->>CLI: peprFormat(opts.validateOnly)
CLI-->>User: success / exit(1)
Reviews (1): Last reviewed commit: "chore: add deprecation warning for eslin..." | Re-trigger Greptile |
| Log.warn( | ||
| "DEPRECATION NOTICE: Pepr v1.1.6 is the last release that supports ESLint v9. " + | ||
| "The next release will upgrade to ESLint v10, which includes breaking changes. " + | ||
| "Review the ESLint v10 migration guide before upgrading Pepr: <ADD_URL_HERE>", |
There was a problem hiding this comment.
Placeholder URL left in warning message
The migration guide URL is still <ADD_URL_HERE>, which will be shown verbatim to every user who runs pepr format. This should be replaced with the actual ESLint v10 migration guide URL before merging (e.g., https://eslint.org/docs/latest/use/migrate-to-10.0.0).
| "Review the ESLint v10 migration guide before upgrading Pepr: <ADD_URL_HERE>", | |
| "Review the ESLint v10 migration guide before upgrading Pepr: https://eslint.org/docs/latest/use/migrate-to-10.0.0", |
Description
Added deprecation warning to
pepr formatto alert users to upcoming breaking changes with eslint 10....
End to End Test:
(See Pepr Excellent Examples)
Related Issue
Fixes #
Relates to #
Type of change
Checklist before merging