chore: bump prettier-linter-helpers to v1.0.1#772
Conversation
🦋 Changeset detectedLatest commit: edbd064 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📝 WalkthroughWalkthroughUpdates Changes
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
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
commit: |
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to ae198ef in 34 seconds. Click for details.
- Reviewed
21lines of code in1files - Skipped
1files when reviewing. - Skipped posting
2draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:66
- Draft comment:
Bump prettier-linter-helpers to ^1.0.1. Confirm the bundled types suffice for TypeScript support. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. package.json:78
- Draft comment:
Remove the now-unnecessary @types/prettier-linter-helpers dev dependency since types are bundled. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_cvuAVczkciFSgQAX
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
ae198ef to
257170e
Compare
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed 257170e in 23 seconds. Click for details.
- Reviewed
32lines of code in2files - Skipped
1files when reviewing. - Skipped posting
2draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .changeset/breezy-steaks-rule.md:1
- Draft comment:
Changeset file correctly marks a patch update. The description clearly states the dependency bump. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. package.json:66
- Draft comment:
Bumped 'prettier-linter-helpers' to ^1.0.1 and removed redundant '@types/prettier-linter-helpers' from devDependencies as types are now bundled. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_eDS1XHPKPhgD0x1T
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (2)
.changeset/breezy-steaks-rule.mdpackage.json
✅ Files skipped from review due to trivial changes (1)
- .changeset/breezy-steaks-rule.md
257170e to
edbd064
Compare
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed edbd064 in 36 seconds. Click for details.
- Reviewed
47lines of code in3files - Skipped
1files when reviewing. - Skipped posting
3draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .changeset/breezy-steaks-rule.md:1
- Draft comment:
Changeset file is correctly formatted with the patch bump for prettier-linter-helpers. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. eslint-plugin-prettier.js:105
- Draft comment:
The refactor in reportDifference uses explicit 'in' checks for insertText and deleteText. Verify that properties that exist but are undefined are handled as intended. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =40%<= threshold50%The comment is asking the PR author to verify that properties that exist but are undefined are handled as intended. This falls under the rule of not asking the author to ensure behavior is intended. However, it does point out a specific area of concern regarding the handling of undefined properties, which could be useful if rephrased to suggest a specific action or ask for confirmation of intention.
3. package.json:65
- Draft comment:
Bumping prettier-linter-helpers version and removing @types/prettier-linter-helpers is appropriate now that types are included. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_bKWmFSorTOxLQnhl
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (3)
.changeset/breezy-steaks-rule.mdeslint-plugin-prettier.jspackage.json
✅ Files skipped from review due to trivial changes (1)
- .changeset/breezy-steaks-rule.md
🚧 Files skipped from review as they are similar to previous changes (1)
- package.json
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
This MR contains the following updates: | Package | Type | Update | Change | OpenSSF | |---|---|---|---|---| | [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node) ([source](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node)) | devDependencies | patch | [`24.10.7` → `24.10.9`](https://renovatebot.com/diffs/npm/@types%2fnode/24.10.7/24.10.9) | [](https://securityscorecards.dev/viewer/?uri=github.com/DefinitelyTyped/DefinitelyTyped) | | [eslint-plugin-prettier](https://github.com/prettier/eslint-plugin-prettier) | devDependencies | patch | [`5.5.4` → `5.5.5`](https://renovatebot.com/diffs/npm/eslint-plugin-prettier/5.5.4/5.5.5) | [](https://securityscorecards.dev/viewer/?uri=github.com/prettier/eslint-plugin-prettier) | --- ### Release Notes <details> <summary>prettier/eslint-plugin-prettier (eslint-plugin-prettier)</summary> ### [`v5.5.5`](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) [Compare Source](prettier/eslint-plugin-prettier@v5.5.4...v5.5.5) ##### Patch Changes - [#​772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@​BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#​776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@​aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue </details> --- - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box --- This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi44NC4yIiwidXBkYXRlZEluVmVyIjoiNDIuODQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGUiXX0=--> See merge request swiss-armed-forces/cyber-command/cea/loom!252 Co-authored-by: Loom MR Pipeline Trigger <group_103951964_bot_9504bb8dead6d4e406ad817a607f24be@noreply.gitlab.com>
chore(deps): update frontend dependencies (patch) (patch) This MR contains the following updates: | Package | Type | Update | Change | OpenSSF | |---|---|---|---|---| | [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node) ([source](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node)) | devDependencies | patch | [`24.10.7` → `24.10.9`](https://renovatebot.com/diffs/npm/@types%2fnode/24.10.7/24.10.9) | [](https://securityscorecards.dev/viewer/?uri=github.com/DefinitelyTyped/DefinitelyTyped) | | [eslint-plugin-prettier](https://github.com/prettier/eslint-plugin-prettier) | devDependencies | patch | [`5.5.4` → `5.5.5`](https://renovatebot.com/diffs/npm/eslint-plugin-prettier/5.5.4/5.5.5) | [](https://securityscorecards.dev/viewer/?uri=github.com/prettier/eslint-plugin-prettier) | --- ### Release Notes <details> <summary>prettier/eslint-plugin-prettier (eslint-plugin-prettier)</summary> ### [`v5.5.5`](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) [Compare Source](prettier/eslint-plugin-prettier@v5.5.4...v5.5.5) ##### Patch Changes - [#​772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@​BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#​776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@​aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue </details> --- - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box --- This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi44NC4yIiwidXBkYXRlZEluVmVyIjoiNDIuODQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGUiXX0=--> See merge request swiss-armed-forces/cyber-command/cea/loom!252 Co-authored-by: Loom MR Pipeline Trigger <group_103951964_bot_9504bb8dead6d4e406ad817a607f24be@noreply.gitlab.com> Co-authored-by: open-source Pipeline <group_90701827_bot_ed04ae348bc5f40af9966fb8b6867e99@noreply.gitlab.com>
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
| datasource | package | from | to | | ---------- | ---------------------- | ----- | ----- | | npm | eslint-plugin-prettier | 5.5.4 | 5.5.5 | ## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555) ##### Patch Changes - [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1 - [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
I pushed a new version, that adds types to the package, so we no longer need
@types/prettier-linter-helpersImportant
Bump
prettier-linter-helpersto v1.0.1, remove@types/prettier-linter-helpers, and updatereportDifference()for optional properties.prettier-linter-helpersto v1.0.1 inpackage.json.@types/prettier-linter-helpersfromdevDependenciesinpackage.json.reportDifference()ineslint-plugin-prettier.jsto handle optionalinsertTextanddeleteTextproperties indifferenceobject.breezy-steaks-rule.mdto document the patch release for the dependency update.This description was created by
for edbd064. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.