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 CDNs for more example JavaScripts #11013

Merged
merged 2 commits into from Oct 10, 2013
Merged

Use CDNs for more example JavaScripts #11013

merged 2 commits into from Oct 10, 2013

Conversation

cvrebert
Copy link
Collaborator

@cvrebert cvrebert commented Oct 9, 2013

Fixes #10989. Using a CDN makes it more explicit that there's an external dependency. It also makes it easier to upgrade stuff, and shaves 2 files off the repo.
/to @twbs/team for review

mdo added a commit that referenced this pull request Oct 10, 2013
Use CDNs for more example JavaScripts
@mdo mdo merged commit 84413f6 into master Oct 10, 2013
@mdo mdo deleted the fix-10989 branch October 10, 2013 03:05
@mdo mdo mentioned this pull request Oct 10, 2013
@mdo mdo restored the fix-10989 branch October 12, 2013 21:00
@mdo mdo deleted the fix-10989 branch October 12, 2013 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Basic template points to files that don't exist
2 participants