-
Notifications
You must be signed in to change notification settings - Fork 12k
Updated global lib story to reflect latest changes required by bootst… #7585
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
Updated global lib story to reflect latest changes required by bootst… #7585
Conversation
configuration - change bootstrap link to point to latest verion - remove tether import from .angular-cli.json - update jquery path in .angular-cli.json to point to jquery.slim.js as per bootstrap documenation - update popper.js path in angular-cli.json to point to use umd verion Closes #7584
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.
Please squash the commits and update the commit message based on the guidelines found here: https://github.com/angular/angular-cli/blob/master/CONTRIBUTING.md#commit
"../node_modules/tether/dist/js/tether.js", | ||
"../node_modules/bootstrap/dist/js/bootstrap.js" | ||
"../node_modules/jquery/dist/jquery.slim.js", | ||
"../node_modules/popper.js/dist/umd/popper.js", |
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.
Indentation seems off. I'd suggest keeping the original two spaces.
configuration for bootstrap4 - update bootstrap url to point to current version - remove tether import from .angular-cli.json and replace it with popper.js - specify popper.js should use umd version Closes #7584
…//github.com/adityaparab/angular-cli into updated-global-lib-page-for-latest-bootstrap
@clydin: I'm having trouble updating commit message and then squashing commits from my machine for some reason. Closing this PR. Will open another one with your review comments implemented. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Will close #7584
…rap4 beta