-
Notifications
You must be signed in to change notification settings - Fork 48
chore(deps-dev): bump tailwindcss from 3.4.19 to 4.1.18 #216
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
Bumps [tailwindcss](https://github.com/tailwindlabs/tailwindcss/tree/HEAD/packages/tailwindcss) from 3.4.19 to 4.1.18. - [Release notes](https://github.com/tailwindlabs/tailwindcss/releases) - [Changelog](https://github.com/tailwindlabs/tailwindcss/blob/main/CHANGELOG.md) - [Commits](https://github.com/tailwindlabs/tailwindcss/commits/v4.1.18/packages/tailwindcss) --- updated-dependencies: - dependency-name: tailwindcss dependency-version: 4.1.18 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Comment |
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.
1 file reviewed, 2 comments
| "prettier": "3.8.0", | ||
| "prettier-plugin-tailwindcss": "^0.7.2", | ||
| "tailwindcss": "^3.4.17", | ||
| "tailwindcss": "^4.1.18", |
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.
logic: Tailwind v4 is a complete rewrite with breaking changes. The current configuration at tailwind.config.ts:1-101 uses v3 syntax which is incompatible with v4. v4 requires migrating to CSS-based configuration using @import "tailwindcss" and @theme directives instead of the JavaScript config file. The @tailwindcss/typography plugin also needs updating for v4 compatibility.
Key breaking changes:
- Config must migrate from JS to CSS-based theme configuration
- Plugin syntax has changed significantly
@tailwindcss/typographyplugin at v0.5.16 may not be compatible with v4tailwindcss-animateplugin needs v4 compatibility verification
Check the official v4 migration guide before merging.
Prompt To Fix With AI
This is a comment left during a code review.
Path: package.json
Line: 96:96
Comment:
**logic:** Tailwind v4 is a complete rewrite with breaking changes. The current configuration at `tailwind.config.ts:1-101` uses v3 syntax which is incompatible with v4. v4 requires migrating to CSS-based configuration using `@import "tailwindcss"` and `@theme` directives instead of the JavaScript config file. The `@tailwindcss/typography` plugin also needs updating for v4 compatibility.
Key breaking changes:
- Config must migrate from JS to CSS-based theme configuration
- Plugin syntax has changed significantly
- `@tailwindcss/typography` plugin at v0.5.16 may not be compatible with v4
- `tailwindcss-animate` plugin needs v4 compatibility verification
Check the official v4 migration guide before merging.
How can I resolve this? If you propose a fix, please make it concise.| "prettier": "3.8.0", | ||
| "prettier-plugin-tailwindcss": "^0.7.2", | ||
| "tailwindcss": "^3.4.17", | ||
| "tailwindcss": "^4.1.18", |
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.
syntax: The lockfile (bun.lockb) is missing from this PR. The CI workflow at .github/workflows/pr-checks.yml:27,49,101 uses bun install --frozen-lockfile which will fail without an updated lockfile.
| "tailwindcss": "^4.1.18", | |
| "tailwindcss": "^4.1.18", |
After making this change, run bun install to update the lockfile and include it in the PR.
Prompt To Fix With AI
This is a comment left during a code review.
Path: package.json
Line: 96:96
Comment:
**syntax:** The lockfile (`bun.lockb`) is missing from this PR. The CI workflow at `.github/workflows/pr-checks.yml:27,49,101` uses `bun install --frozen-lockfile` which will fail without an updated lockfile.
```suggestion
"tailwindcss": "^4.1.18",
```
After making this change, run `bun install` to update the lockfile and include it in the PR.
How can I resolve this? If you propose a fix, please make it concise.
Bumps tailwindcss from 3.4.19 to 4.1.18.
Release notes
Sourced from tailwindcss's releases.
... (truncated)
Changelog
Sourced from tailwindcss's changelog.
Commits
9b32f7cRelease v4.1.18 (#19431)820d907ExposecandidatesToAstto the language server (#19405)478e959Don’t emit color-mix fallback rules inside@keyframes(#19419)a5f4644Validate named values in candidate parser (#19397)229121dCanonicalization: combinetext-*andleading-*classes (#19396)243615eHandle backwards compatibility forcontenttheme from JS configs (#19381)7642751Improve compatibility with special default values in JS configs (#19348)af48117remove unnecessary intermediate check9e436f7Try to canonicalize any arbitrary utility to a bare value (#19379)479b725Bump Vitest to v4 (#19216)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Greptile Summary
This PR upgrades
tailwindcssfrom v3.4.19 to v4.1.18, a major version bump that introduces significant breaking changes.Critical Issues:
tailwind.config.ts) to CSS-based configuration using@themedirectivespackage.jsonwithout updatingbun.lockb, which will cause CI failures at.github/workflows/pr-checks.yml:27,49,101where--frozen-lockfileis used@tailwindcss/typographyv0.5.16 andtailwindcss-animatev1.0.7, both need verification for v4 compatibility@tailwind base/components/utilitiesatsrc/app/globals.css:1-3and JavaScript config) which is incompatible with v4's new engineRequired migration work:
tailwind.config.tsfrom JavaScript to CSS-based@themeconfigurationbun installThis is not a drop-in replacement - it requires careful migration and testing.
Confidence Score: 0/5
Important Files Changed
Sequence Diagram
sequenceDiagram participant D as Dependabot participant P as package.json participant L as Lockfile participant C as CI/CD participant T as Tailwind Config participant B as Build Process D->>P: Bump tailwindcss 3.4.19→4.1.18 Note over P: Major version change P--xL: ❌ Lockfile not updated Note over L: Missing in PR C->>L: bun install --frozen-lockfile L--xC: ❌ Fails - lockfile out of sync B->>T: Read tailwind.config.ts Note over T: v3 JavaScript config T--xB: ❌ Incompatible with v4 Note over T: v4 requires CSS config B->>P: Load plugins Note over P: `@tailwindcss/typography` v0.5.16<br/>`tailwindcss-animate` v1.0.7 P--xB: ❌ Plugin compatibility unknown Note over D,B: PR will fail CI checks<br/>Requires migration work