Skip to content
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

Drop css-minimizer-webpack-plugin 5 support, only css-minimizer-webpack-plugin 7 is supported #1320

Closed

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented Sep 1, 2024

@Kocal Kocal added this to the 5.0 milestone Sep 1, 2024
@Kocal
Copy link
Member Author

Kocal commented Sep 1, 2024

Failing checks due to Node.21 will be fixed by #1321, more work is needed for the others.

@stof
Copy link
Member

stof commented Sep 2, 2024

This requires bumping the min node version from 18.0.0 to 18.12.0 due to their BC break in 6.0

@stof
Copy link
Member

stof commented Sep 2, 2024

The failures are probably not directly related to this PR. #818 fails the "low dependencies" jobs while it only changes comments.

@stof
Copy link
Member

stof commented Sep 2, 2024

I confirm that a build in the master branch also breaks there: https://github.com/symfony/webpack-encore/actions/runs/10666578054/job/29562308888

And #1322 seems to fix it.

This PR was merged into the main branch.

Discussion
----------

Drop support for Node.js 19 and 21

Due to the `>=18.0.0` version constraint in `engines.node`, it means that we allows Node.js 19 and 21, but, we don't want that.

Odds versions are supported only 6 months, it only helps library maintainers to update their library comptability with a new Node.js LTS-version. You can see on https://nodejs.org/fr/about/previous-releases that Node.js 19 is not present and Node.js 21 support has been stopped around ~may 2024.

Some of our dependencies explicitly does not support explicitly Node.js 19/21, like css-minimizer-webpack-plugin that [I've tried to upgrade to v5](https://github.com/symfony/webpack-encore/actions/runs/10654621974/job/29531168804?pr=1320):
```
yarn install v1.22.22
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
error cssnano@7.0.5: The engine "node" is incompatible with this module. Expected version "^18.12.0 || ^20.9.0 || >=22.0". Got "21.7.3"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
```

Since we plan to release a new major version of Encore, it's the good moment.

WDYT?

Commits
-------

40ce3c9 Drop support for Node.js 19 and 21
@Kocal Kocal force-pushed the chore/css-minimizer-webpack-plugin-7 branch from 7b4bcf9 to 9658190 Compare September 2, 2024 15:13
@Kocal Kocal closed this Sep 2, 2024
Kocal added a commit that referenced this pull request Sep 2, 2024
…inimizer-webpack-plugin 7 is supported (Kocal)

This PR was merged into the main branch.

Discussion
----------

Drop css-minimizer-webpack-plugin 5 support, only css-minimizer-webpack-plugin 7 is supported

Replace #1320

It also upgrade cssnano to ^7, see https://github.com/webpack-contrib/css-minimizer-webpack-plugin/blob/master/CHANGELOG.md#700-2024-05-07

Node.js support has been updated to ^18.12

Commits
-------

26141e3 Drop css-minimizer-webpack-plugin 5 support, only css-minimizer-webpack-plugin 7 is supported
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