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

#21 Scoped @uirouter fix #22

Closed
wants to merge 3 commits into from
Closed

Conversation

lenin-anzen
Copy link
Contributor

Fixes the warning #21

Just change:
angular-ui-router

To:
@uirouter/angularjs

lenin-anzen and others added 3 commits June 28, 2017 13:02
The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/npm:tunnel-agent:20170305

Latest report for lenin-anzen/generator-angular-webpack-es6:
https://snyk.io/test/github/lenin-anzen/generator-angular-webpack-es6
[Snyk Update] New fixes for 1 vulnerable dependency path
@STUkh
Copy link
Owner

STUkh commented Aug 15, 2017

I have runtime errors after applying this PR. Could you re-check and prove that everything is smooth?

@STUkh
Copy link
Owner

STUkh commented Sep 16, 2017

Can't run on Mac with new yeoman package version.
Reverted yeoman version and made merge with your ui-router package name changes. Thanks for noticing that.

@STUkh STUkh closed this Sep 16, 2017
STUkh added a commit that referenced this pull request Sep 16, 2017
# Conflicts:
#	package-lock.json
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.

2 participants