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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade JS dependencies #9062

Merged
merged 4 commits into from Apr 13, 2017
Merged

Conversation

Igloczek
Copy link
Contributor

@Igloczek Igloczek commented Mar 30, 2017

Upgrade of all of JS dependencies required to run Grunt.js task. Some packages have BC due to major releases, so I had to adjust configuration.

I also changed accepted version range from ^ to ~, b/c it's safer to use only hotfix upgrades, than minor (extremely important when we are not maintaining any lockfile for these dependencies).

Some tasks don't work, b/c was broken before changes i.e. grunt styledocco:documentation.
Some of them work just weird like grunt replace:documentation, b/c on every run they add license comments, even it already are added, but also it's not related to packages upgrade.

Nice things - styles compilation works faster 馃槈

@ishakhsuvarov ishakhsuvarov added this to the April 2017 milestone Apr 11, 2017
@magento-team magento-team merged commit a1918a5 into magento:develop Apr 13, 2017
magento-team pushed a commit that referenced this pull request Apr 13, 2017
@magento-team
Copy link
Contributor

@Igloczek thank you for your contribution to Magento 2 project

@Igloczek Igloczek deleted the update-js-dependencies branch April 14, 2017 19:17
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.

None yet

4 participants