Skip to content

fix: reduce npm install deprecation warnings#47

Open
Akanksha-020 wants to merge 3 commits into
snackoverflowasad:mainfrom
Akanksha-020:fix/install-deprecation-warnings
Open

fix: reduce npm install deprecation warnings#47
Akanksha-020 wants to merge 3 commits into
snackoverflowasad:mainfrom
Akanksha-020:fix/install-deprecation-warnings

Conversation

@Akanksha-020

@Akanksha-020 Akanksha-020 commented May 25, 2026

Copy link
Copy Markdown

Description of Changes

This PR reduces the deprecation warnings shown during npm install by updating the dependency tree with safe npm overrides and modernizing the lint setup.

Specifically:

  • Bumped google-auth-library through an override to remove the deprecated uuid@9 chain from the Google auth stack.
  • Bumped file-entry-cache and glob through overrides to remove the deprecated rimraf@3 and glob@7 ESLint cache chain.
  • Upgraded ESLint to 10.x and replaced the legacy config with a flat eslint.config.js so the project continues linting cleanly on the newer toolchain.

This keeps the bot building and testing normally while making install output cleaner for new contributors.

closes #28

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Refactor or other maintenance

How Was This Tested?

I verified the changes with:

  • npm install
  • npm run lint
  • npm run build
  • npm test

All of the above passed after the dependency and config updates.

Checklist

  • The code builds without errors (npm run build).
  • Existing tests pass (npm test).
  • New tests have been added if needed.
  • Documentation has been updated to reflect these changes if needed.

@Akanksha-020

Copy link
Copy Markdown
Author

hey i raised the pr yesterday for issue #28 olease review and see if any changes are required!

@snackoverflowasad

Copy link
Copy Markdown
Owner

Can you share a screen recording or anything it'll be easy for me to evaluate and also the checks are not passed please check it once

@snackoverflowasad

Copy link
Copy Markdown
Owner

hey @Akanksha-020 please check the build are failing

@Akanksha-020

Copy link
Copy Markdown
Author

Ok I will review the issues and fix it and then request a review again when all the checks are passing

@snackoverflowasad

Copy link
Copy Markdown
Owner

REMINDER TO THIS PR @Akanksha-020

@shouri123

Copy link
Copy Markdown
Collaborator

@Akanksha-020 there aree 11 merge conflicts fix them

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.

[BUG] chore: resolve npm install deprecation warnings from dependencies

3 participants