Skip to content

Comments

chore(deps): clean up unused root deps and fix dep categories#112

Merged
rlorenzo merged 2 commits intomainfrom
security-package-updates
Feb 21, 2026
Merged

chore(deps): clean up unused root deps and fix dep categories#112
rlorenzo merged 2 commits intomainfrom
security-package-updates

Conversation

@rlorenzo
Copy link
Contributor

  • Remove unused eslint-config-prettier and typescript-eslint from root (not referenced in eslint.config.mjs), add eslint explicitly
  • Move glob from dependencies to devDependencies in VueApp

Fixes the Jenkins error.

- Remove unused eslint-config-prettier and typescript-eslint from
  root (not referenced in eslint.config.mjs), add eslint explicitly
- Move glob from dependencies to devDependencies in VueApp
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR cleans up the root package dependencies by removing packages that are not directly referenced in the root eslint.config.mjs file, adds the eslint package explicitly, and corrects the categorization of the glob package in VueApp.

Changes:

  • Removed eslint-config-prettier and typescript-eslint from root devDependencies
  • Added eslint explicitly to root devDependencies
  • Moved glob from dependencies to devDependencies in VueApp

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

File Description
package.json Removed unused eslint-config-prettier and typescript-eslint; added eslint explicitly for @eslint/js usage
package-lock.json Updated lock file to reflect root package.json changes; removed lint-staged (custom scripts used instead)
VueApp/package.json Moved glob to devDependencies (correctly categorized as dev-only)
VueApp/package-lock.json Updated lock file with glob recategorization and various dependency version updates
Files not reviewed (1)
  • VueApp/package-lock.json: Language not supported

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • VueApp/package-lock.json: Language not supported

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • VueApp/package-lock.json: Language not supported

@rlorenzo
Copy link
Contributor Author

@bsedwards, I hope you don't mind me just merging this in without approval. It is a package update and would block deployment for prod if it isn't merged in.

@rlorenzo rlorenzo merged commit d4feba8 into main Feb 21, 2026
13 checks passed
@rlorenzo rlorenzo deleted the security-package-updates branch February 21, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants