-
Notifications
You must be signed in to change notification settings - Fork 67
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 #76
Comments
This factors very much into the larger scale effort. For now I'd focus on getting your design refactor done, then we can focus on some of the behind the scenes work that makes the machine run. |
@bmuenzenmeyer Yep I figured as much. Just flagging it since I just read the post. 👍 |
@bmuenzenmeyer @bradfrost @EvanLovely FWIW, I already killed off bower in the refactor I started working on 3 months ago (in addition to chipping away at getting the JS to compile with Webpack, breaking apart the JS into more standalone modules that manage their own includes without a full JS refactor (which we really do need), getting all the JS script tags in PL to load up asynchronously without being dependent on a particular loading order, etc). There's just a couple minor things that I never got around to on the Drupal Pattern Lab fork (namely just cleaning up some of the bower configs that are no longer used + reverting the hogan.js templating for the navigation since this branch also includes me getting the main PL menu to render with Twig templating](https://github.com/drupal-pattern-lab/styleguidekit-assets-default/blob/feature/modernize-patternlab-js/src/scripts/modules/layout/index.js) vs Mustache). |
Sounds cool @sghoweri - sounds like a trail is already blazed by your hard work ! I think a good time to tackle this is once other features settle in - this is mostly backend convenience and doesn't affect users much |
@bmuenzenmeyer @EvanLovely trying to get my ducks in a row on this (busy week!) but for what it’s worth I think I’ve got the work needed to be done on this already figured out for a while now, just need to get the PR specific bits to this put together (and likely fold in the build tools updates at the same time given the overlapping dependencies): https://github.com/bolt-design-system/bolt/tree/epic/v2/packages/styleguidekit-assets-bolt |
What to do about this? https://bower.io/blog/2017/how-to-migrate-away-from-bower/
Cc @bmuenzenmeyer
The text was updated successfully, but these errors were encountered: