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

Use unminified file in "main" in bower.json #302

Merged
merged 2 commits into from Sep 4, 2014
Merged

Use unminified file in "main" in bower.json #302

merged 2 commits into from Sep 4, 2014

Conversation

diagramatics
Copy link
Contributor

As per Bower's commands.

Side note: I've been thinking of including textAngular-sanitize.js but it's not really mandatory. Is there any way to actually use angular-sanitize.js and override the usage of it? That way we can include angular-sanitize.js to the dependency and add the overrides to the main (or add them to textAngular.js).

[As per Bower's commands](https://github.com/bower/bower.json-spec#main).

Side note: I've been thinking of including `textAngular-sanitize.js` but it's not really mandatory. Is there any way to actually use `angular-sanitize.js` and override the usage of it? That way we can include `angular-sanitize.js` to the dependency and add the overrides to the main (or add them to `textAngular.js`).
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e555be7 on diagramatics:patch-1 into 94545cb on fraywing:master.

Didn't notice textAngular.min.js was a minified version of the combined two.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 0036ba3 on diagramatics:patch-1 into 94545cb on fraywing:master.

@SimeonC
Copy link
Collaborator

SimeonC commented Sep 4, 2014

Thanks for pointing this out. I'll merge it as is for now.

As for the textAngular-sanitize, I think that if you load angular-sanitize, then textAngular-sanitize and they load and execute in that order then textAngular-sanitize should overwrite angular-sanitize.

On the other hand, I'm awaiting this: angular/angular.js#5900 so hopefully we can do away with textAngular-sanitize altogether. When I get time I may attempt to help out on that, but it is slated for the next milestone so one can hope!

SimeonC pushed a commit that referenced this pull request Sep 4, 2014
fix(bower) Use unminified file in "main" in bower.json as per bower spec
@SimeonC SimeonC merged commit 91d74bf into textAngular:master Sep 4, 2014
@diagramatics diagramatics deleted the patch-1 branch September 7, 2014 02:49
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

3 participants