-
Notifications
You must be signed in to change notification settings - Fork 88
Update glob and related packages #430
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 upgrades the project from Tailwind CSS v3.4.16 to v4.1.17, a major version upgrade that requires changes to the PostCSS configuration and introduces new dependencies. The upgrade modernizes the CSS tooling stack to use Tailwind CSS v4's new architecture.
Key Changes:
- Updated Tailwind CSS from v3.4.16 to v4.1.17 (major version upgrade)
- Switched PostCSS configuration to use the new
@tailwindcss/postcssplugin instead of the directtailwindcssintegration - Added
globpackage as a new dependency for file pattern matching
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
postcss.config.js |
Updated PostCSS plugin configuration to use @tailwindcss/postcss instead of tailwindcss to align with Tailwind CSS v4's new plugin architecture |
package.json |
Upgraded tailwindcss to v4.1.17, added @tailwindcss/postcss as a dev dependency, and added glob package |
package-lock.json |
Updated lock file reflecting the new dependency tree for Tailwind CSS v4, including new packages like @tailwindcss/oxide, lightningcss, and platform-specific binaries |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this 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 6 out of 10 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this 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 8 out of 12 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "sass": "^1.94.2", | ||
| "slugify": "^1.6.6", | ||
| "typescript": "^5.7.3" | ||
| "typescript": "^5.9.3" |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TypeScript has been downgraded from ^5.7.3 to ^5.9.3, which is actually an older version (5.9 < 5.7). This appears to be unintentional. Consider updating to the latest TypeScript 5.7.x version to benefit from the latest features and bug fixes.
| "typescript": "^5.9.3" | |
| "typescript": "^5.7.3" |
| @@ -1,6 +1,6 @@ | |||
| type IndexedTopics = Record<string, string>; | |||
| type IndexedTopicsType = Record<string, string>; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made this change to resolve ts lint error.
'IndexedTopics' is already defined @typescript-eslint/no-redeclare
This pull request introduces several updates focused on upgrading dependencies, modernizing tooling, and improving code quality and maintainability. The most significant changes include major dependency upgrades (notably React, Next.js, and FontAwesome), migration to a new ESLint configuration system, and updates to the Node.js version used in CI workflows.
Dependency and Tooling Upgrades:
.github/workflows/deploy.ymland.github/workflows/pr.yml). [1] [2]package.jsonto their latest major versions, includingreact,next,@fortawesomepackages,typescript,eslint, and more. Added new dependencies such asgloband@tailwindcss/postcss.Linting Configuration Migration:
.eslintrc.jsonconfiguration file with a neweslint.config.mjsfile, migrating to the flat config format. This includes updated plugin usage, custom rules, and ignore patterns. [1] [2]lintscript inpackage.jsonto use the new ESLint command instead of the Next.js lint command.Build and PostCSS Configuration:
postcss.config.jsto use the new@tailwindcss/postcssplugin instead oftailwindcss.Minor Code Improvements:
generate.tsandtopics.tsfor improved type safety and clarity. [1] [2] [3] [4]