Skip to content

CDN Tab [NOT READY]#41

Closed
jdorfman wants to merge 5 commits intojquery:masterfrom
jdorfman:maxcdn
Closed

CDN Tab [NOT READY]#41
jdorfman wants to merge 5 commits intojquery:masterfrom
jdorfman:maxcdn

Conversation

@jdorfman
Copy link
Contributor

  • Created a new MD file and moved everything regarding CDN from download.md to cdn.md
  • Made the urls protocol-less since MaxCDN will provide SSL
  • Also added JSDelivr.com to the other CDNs section.

@scottgonzalez
Copy link
Member

What's the deal with the symlink?

@jdorfman
Copy link
Contributor Author

@scottgonzalez I apologize I forgot to git rm it. it was so I can easily get to jquery-wp-content to edit themes/jquery/menu-header.php want me to do a new PR?

@scottgonzalez
Copy link
Member

No need for a new PR. We can clean up the commits when we're ready to merge.

@gnarf
Copy link
Member

gnarf commented Jul 10, 2013

You can just add another commit and or amend the one you have. once you push it will update the pull request

@jdorfman
Copy link
Contributor Author

@scottgonzalez @gnarf done thanks again sorry

@scottgonzalez
Copy link
Member

You'll need to actually revert those two commits. We don't want it gitignored; it needs to just not exist.

@scottgonzalez
Copy link
Member

We'll still need to mention CDNs and past releases on the download page.

@jdorfman
Copy link
Contributor Author

ok my bad, I will do it later or first thing tomorrow.

@jdorfman jdorfman closed this Jul 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants