-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Migrate away from bower workaround to npm #31968
Comments
@PVince81 - please include devops in the process ( with the eye of release automation ) |
let me throw this POC back into the arena: #30088 |
@PVince81 reopen & rebase to see how far we get? |
rebased and conflicts resolved - lets see what we want/can reuse |
@felixheidecke offered to help. We need to solve the dependency order issue, see #30088 (comment) One idea is to locally do a manual concat with grunt and see if we observe the same result. If yes it would confirm that the problem is related to loading order and not something else. |
@PVince81 @DeepDiver1975 I found a couple of duplicates such as:
As well as two libs that do (essentially) the same thing:
Since strengthify doesn't have an npm package, and no package.json file (can't reference the git path either) I'll drop it. I' sticking to the latest version in its major release present in the package file. |
@felixheidecke are you working on the actual package.json or the updated one ? Our bundling problem is on this PR #30088 where I believe we already removed duplicate dependencies. zxcvbn is the database used for strengthify. |
I'm working on the actual package also pulling from the other PR. |
Please only work on the PR as references above |
@DeepDiver1975 I tried a different approach using grunt.js instead of webpack because it gives us more control as it simply concatenates files in the desired order. Also works on CSS files as well as modules and CSS found in core. I recommend that approach over webpack. |
whatever you do it won't help with the problem at hand the problem is that there is a forced update of jquery.ui from 1.10.0 to 1.10.4 and this is what breaks it. so this is blocked by #18739 or we could spend some time fixing jquery-ui-fixes.css to work with 1.10.4... but I don't think it's worth it if it's just for the bundling part |
backlog for now, blocked by jquery.ui update/removal |
jquery.ui updates still in progress... and still blocked |
we will not invest more into this -> close |
A few releases ago we moved away from the bower file and moved the dependencies into the build/package.json file for npm. However they still use the "@bower_components" approach which is designed to be transitional. Many automation tools have trouble with these.
We should finish the transition to full npm dependencies and find a way to package said dependencies, maybe with webpack.
Some work had already started in the first commits of #30166 but we got stuck due to weird issues with select2, possibly related to loading order.
Estimate: 2-3md as we need to debug and understand the weird issue and also hope there aren't more such issues.
@DeepDiver1975 @felixheidecke
The text was updated successfully, but these errors were encountered: