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

feat(cdnify): support 'all' flag in cdn data #21

Merged
merged 1 commit into from
Mar 7, 2014
Merged

feat(cdnify): support 'all' flag in cdn data #21

merged 1 commit into from
Mar 7, 2014

Conversation

shahata
Copy link
Collaborator

@shahata shahata commented Mar 7, 2014

'all' flag means that the cdn has all the files in this bower component and that the script tag in the html document should be updated no matter what. in this case, we don't go to bower to get the component's main file name. this is important for components like angular-i18n, where there are many files the developer might include and there is no single 'main file'.

'all' flag means that the cdn has all the files in this bower component and that the script tag in the html document should be updated no matter what. in this case, we don't go to bower to get the component's main file name.
@passy
Copy link
Owner

passy commented Mar 7, 2014

The semantics of the main property are likely to change in the future, but until then this is probably a good workaround.

passy added a commit that referenced this pull request Mar 7, 2014
feat(cdnify): support 'all' flag in cdn data
@passy passy merged commit acda2b3 into passy:master Mar 7, 2014
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