chore: bump prettier-linter-helpers to v1.0.1 by BPScott · Pull Request #772 · prettier/eslint-plugin-prettier (original) (raw)
📝 Walkthrough
Walkthrough
Updates prettier-linter-helpers dependency to ^1.0.1, removes @types/prettier-linter-helpers from devDependencies, adds a changeset for a patch release, and adjusts reportDifference in eslint-plugin-prettier.js to more explicitly handle missing operation fields when computing insert/delete text and ranges.
Changes
| Cohort / File(s) | Summary |
|---|---|
| Dependency updates package.json | Bumped prettier-linter-helpers from ^1.0.0 → ^1.0.1; removed @types/prettier-linter-helpers from devDependencies. |
| Release metadata .changeset/... .changeset/breezy-steaks-rule.md | Added changeset entry marking a patch release for eslint-plugin-prettier and noting the prettier-linter-helpers bump to 1.0.1. |
| Logic change eslint-plugin-prettier.js | reportDifference rewritten to extract operation and offset first and to conditionally define insertText / deleteText, avoiding destructuring defaults and making absent fields explicit before computing ranges. |
Sequence Diagram(s)
(omitted — changes are small and do not introduce a multi-component sequential flow)
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
- feat: support non-js languages #743 — Related edits to
reportDifferenceimplementation ineslint-plugin-prettier.js.
Suggested reviewers
- JounQin
Poem
🐰 A nibble at the deps tonight,
I hop and tweak till values fit just right.
Types waved off, a tiny bump we send,
A changeset tucked in — tidy end. ✨
Pre-merge checks and finishing touches
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title 'chore: bump prettier-linter-helpers to v1.0.1' accurately summarizes the main change in the pull request, which is updating the prettier-linter-helpers dependency to version 1.0.1. |
| Docstring Coverage | ✅ Passed | Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. |
✨ Finishing touches
- 📝 Generate docstrings 🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
prettier-linter-helpers-bump
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.