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

Annotate directive to avoid issues with minification #18

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

Conversation

keathley
Copy link

@keathley keathley commented May 6, 2014

The array syntax is needed to avoid injecting errors after the directive has been minified. This fix should solve those issues. Thanks in advance for any feedback that you have.

@emwardell
Copy link

Thanks for the fix. How does one get this fix into the bower package?

@keathley
Copy link
Author

@emwardell This pull request will have to be merged or you can specify my fork with the commit sha in your bower.json file for the time being (that's what I'm currently doing).

@ghost
Copy link

ghost commented Aug 2, 2014

@btford Can you please merge this, or one of the other 3 PR's that have been opened to fix this issue? Thanks.

@keathley
Copy link
Author

keathley commented Aug 4, 2014

@dave-elkan Since there are 3 open pull requests for the same issue I'm not sure if there's a plan to pull any of them in. In the meantime you can use my fork (or one of the others).

@ghost
Copy link

ghost commented Aug 4, 2014

Thanks @spyc3r. I did just that the other day - I was just hoping the author would do his house-keeping as well though. 👍

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