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.

Sign up for GitHub

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

Jump to bottom

fix DI #32

Closed

ts-ign0re wants to merge 1 commit into vinaygopinath:master from ts-ign0re:master

ts-ign0re commented Feb 28, 2017

The problem is:

import ngMeta from 'ng-meta'; // error here - unused variable
const app = angular.module(APP_NAME, ['ngMeta']);

in this case we have an eslint error, but if we inject ngMeta as object we will have injection error (lost dependency name on minification)

Author

ts-ign0re commented Feb 28, 2017

@vinaygopinath all tests is ok


          fix DI

d08f746

ts-ign0re closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment