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

[Task] Update webpack #211

Merged
merged 6 commits into from
Dec 2, 2024
Merged

[Task] Update webpack #211

merged 6 commits into from
Dec 2, 2024

Conversation

mattamon
Copy link
Contributor

@mattamon mattamon commented Nov 5, 2024

Resolves #201
Update to 1.17 possible
Restricted due to pimcore/admin-ui-classic-bundle

 - Root composer.json requires pimcore/admin-ui-classic-bundle * -> satisfiable by pimcore/admin-ui-classic-bundle[dev-722-fix-manual-sorting, 1.6.x-dev].
 - pimcore/admin-ui-classic-bundle 1.6.x-dev requires symfony/webpack-encore-bundle ^1.13.2 -> satisfiable by symfony/webpack-encore-bundle[v1.13.2, ..., v1.17.2].

@mattamon mattamon added this to the 1.3.0 milestone Nov 5, 2024
composer.json Outdated
@@ -20,7 +20,7 @@
"pimcore/personalization-bundle": "^1.0",
"pimcore/google-marketing-bundle": "^1.0",
"knplabs/knp-paginator-bundle": "^6.0.0",
"symfony/webpack-encore-bundle": "^1.13.2",
"symfony/webpack-encore-bundle": "^1.17",
Copy link
Contributor

Choose a reason for hiding this comment

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

@mattamon I think we could support both versions. Also in admin-ui-classic-bundle. WDYT?

Suggested change
"symfony/webpack-encore-bundle": "^1.17",
"symfony/webpack-encore-bundle": "^1.13.2 || ^2.0",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah but I would at least already set it to 1.17 so like ^1.17 || ^2.0

@blankse
Copy link
Contributor

blankse commented Nov 26, 2024

@mattamon symfony/webpack-encore-bundle < 1.13.2 can't be installed. So it makes no sense to set it lower.

composer.json Outdated Show resolved Hide resolved
Co-authored-by: Jacob Dreesen <j.dreesen@neusta.de>
@mattamon
Copy link
Contributor Author

@mattamon symfony/webpack-encore-bundle < 1.13.2 can't be installed. So it makes no sense to set it lower.

yeah sorry it was a typo

Copy link

sonarcloud bot commented Nov 29, 2024

@kingjia90
Copy link
Contributor

Mhhh, the tests are false positive, because it's still installing the v1, due to personalization bundle which requires an admin ui bundle that doesn't have pimcore/admin-ui-classic-bundle#770 yet

@mattamon mattamon merged commit 5ea8f15 into 1.x Dec 2, 2024
9 checks passed
@mattamon mattamon deleted the 201-upgrade-webpack branch December 2, 2024 13:02
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement] Upgrade symfony/webpack-encore-bundle
4 participants