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 cdn.rawgithub and Link To Specific Version Rather Than Master #274

Merged
merged 4 commits into from
Apr 18, 2015

Conversation

svozza
Copy link
Collaborator

@svozza svozza commented Apr 17, 2015

Fixes #205

I don't really know how the doc generation works but is there a way that we can add the version of Highland dynamically? Something like:

<script src="https://cdn.rawgit.com/caolan/highland/{{version}}/dist/highland.js" type="text/javascript"></script

Otherwise I can see us forgetting to update this file occasionally.

@svozza svozza modified the milestone: v2.5.0 Apr 17, 2015
fix other links to use cdn

fix stream doc info for sort functions

fix javadocs
@svozza svozza force-pushed the fix-raw-cdn-issue branch from 88766d0 to 3a6d708 Compare April 17, 2015 15:32
@vqvu
Copy link
Collaborator

vqvu commented Apr 17, 2015

Yeah, this is possible. We use handlebars, so we just need to add the proper value into the context.

You can pull in vqvu@d40b335 (docs-gen branch) and run the pre-release:* tasks to test.

@svozza
Copy link
Collaborator Author

svozza commented Apr 17, 2015

For some reason I couldn't cherry pick that commit so I just copied out the changes manually as there weren't many of them. The pre-release task worked for me locally and all the links were correct.

@vqvu
Copy link
Collaborator

vqvu commented Apr 17, 2015

You should undo the package.json and bower.json changes. Those should only be updated once we release.

@svozza
Copy link
Collaborator Author

svozza commented Apr 17, 2015

Doh! That's what I get for being lazy and doing git commit -am.

@vqvu
Copy link
Collaborator

vqvu commented Apr 17, 2015

LGTM

svozza added a commit that referenced this pull request Apr 18, 2015
Use cdn.rawgithub and Link To Specific Version Rather Than Master
@svozza svozza merged commit a98cd28 into caolan:master Apr 18, 2015
@svozza svozza deleted the fix-raw-cdn-issue branch April 18, 2015 06:44
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.

Source link in docs should point to correct commit.
2 participants