Skip to content

Conversation

@pavankjadda
Copy link
Contributor

No description provided.

@pavankjadda pavankjadda requested a review from Copilot October 15, 2025 20:11
@pavankjadda pavankjadda self-assigned this Oct 15, 2025
@pavankjadda pavankjadda added the dependencies Pull requests that update a dependency file label Oct 15, 2025
@netlify
Copy link

netlify bot commented Oct 15, 2025

Deploy Preview for js-react-kit failed. Why did it fail? →

Name Link
🔨 Latest commit 7629f6f
🔍 Latest deploy log https://app.netlify.com/projects/js-react-kit/deploys/68f15e5a68701d00086ecda0

@nx-cloud
Copy link

nx-cloud bot commented Oct 15, 2025

View your CI Pipeline Execution ↗ for commit 84ed589

Command Status Duration Result
nx test react-kit ✅ Succeeded 9s View ↗
nx build react-kit ✅ Succeeded 5s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-15 20:12:53 UTC

Copy link
Contributor

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 updates several dependencies and devDependencies in package.json to improve stability and align with newer toolchain versions.

  • Bumps core tooling (Nx, TypeScript, Vite) and UI stack (MUI, React Router).
  • Moves to newer ESLint TypeScript tooling and OIDC client.

"@types/react-dom": "19.1.3",
"@typescript-eslint/eslint-plugin": "8.46.1",
"@typescript-eslint/parser": "8.46.1",
"@vitejs/plugin-react": "^4.4.1",
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

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

@vitejs/plugin-react 4.x typically declares a peer dependency range that does not include Vite 7, which will cause peer dependency warnings or build issues. Please upgrade the plugin to a version compatible with Vite 7 (e.g., "@vitejs/plugin-react": "^5.x") to match the Vite 7 toolchain.

Suggested change
"@vitejs/plugin-react": "^4.4.1",
"@vitejs/plugin-react": "^5.0.0",

Copilot uses AI. Check for mistakes.
"sass": "^1.87.0",
"semantic-release": "^24.2.3",
"typescript": "5.9.3",
"vite": "7.1.9",
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

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

@vitejs/plugin-react 4.x typically declares a peer dependency range that does not include Vite 7, which will cause peer dependency warnings or build issues. Please upgrade the plugin to a version compatible with Vite 7 (e.g., "@vitejs/plugin-react": "^5.x") to match the Vite 7 toolchain.

Copilot uses AI. Check for mistakes.
Comment on lines +68 to +69
"typescript": "5.9.3",
"vite": "7.1.9",
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

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

[nitpick] Typescript and Vite are now pinned to exact versions while most other entries use semver ranges (^). Consider aligning on a consistent versioning strategy (either pin both all dev tools or use caret ranges) to make updates predictable and reduce inadvertent drift.

Suggested change
"typescript": "5.9.3",
"vite": "7.1.9",
"typescript": "^5.9.3",
"vite": "^7.1.9",

Copilot uses AI. Check for mistakes.
@pavankjadda pavankjadda merged commit 5901aa0 into main Oct 16, 2025
1 of 5 checks passed
@github-actions
Copy link

🎉 This PR is included in version 5.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

dependencies Pull requests that update a dependency file released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants