Skip to content

chore(bot): bump dependencies#1728

Merged
github-actions[bot] merged 1 commit intomainfrom
create-pull-request/patch-1776249133
Apr 15, 2026
Merged

chore(bot): bump dependencies#1728
github-actions[bot] merged 1 commit intomainfrom
create-pull-request/patch-1776249133

Conversation

@donniean
Copy link
Copy Markdown
Owner

Automated changes by create-pull-request GitHub action

Signed-off-by: github-actions[bot] <12584040+donniean@users.noreply.github.com>
@github-actions github-actions bot enabled auto-merge (squash) April 15, 2026 10:32
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 15, 2026

⚠️ No Changeset found

Latest commit: e1f5bba

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot merged commit 953fa03 into main Apr 15, 2026
9 checks passed
@github-actions github-actions bot deleted the create-pull-request/patch-1776249133 branch April 15, 2026 10:33
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the prettier dependency to version 3.8.3 and synchronizes the lockfile, which also includes updates for baseline-browser-mapping, get-tsconfig, and std-env. Feedback was provided regarding the redundant listing of prettier in both dependencies and devDependencies, suggesting it should be consolidated to the appropriate section based on whether it is a runtime or development-only dependency.

Comment thread package.json
"lint-staged": "16.4.0",
"markdownlint-cli": "0.48.0",
"prettier": "3.8.2",
"prettier": "3.8.3",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The prettier dependency is redundantly listed in both dependencies (line 72) and devDependencies. If prettier is used programmatically by the application at runtime, it should be removed from devDependencies. If it is only used for development tasks (such as the linting scripts), it should be moved entirely to devDependencies and removed from dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant