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

To be minified and compressed without errors that cased by changing t… #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

faressoft
Copy link

To be minified and compressed without errors that cased by changing the parameter $rootScope to an auto-naming parameter like 'a' which will not be recognised my the angular.js as $rootScope any more.

…he parameter to an auto-naming parameter like 'a' which will not be recognised my the angular.js as $rootScope any more.
@ChrisMBarr
Copy link

Yes please! This is a very simple PR that we need. Currently we use the sctrictDI: true in our app, and the app fails to load due to this right now. We'd like to be able to get this code from NPM, but instead we have to manually include it in our source and made this modification so it will work.

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

2 participants