chore(bot): bump dependencies#1728
Conversation
Signed-off-by: github-actions[bot] <12584040+donniean@users.noreply.github.com>
|
There was a problem hiding this comment.
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.
| "lint-staged": "16.4.0", | ||
| "markdownlint-cli": "0.48.0", | ||
| "prettier": "3.8.2", | ||
| "prettier": "3.8.3", |
There was a problem hiding this comment.
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.
Automated changes by create-pull-request GitHub action