Skip to content

Conversation

jacobstr
Copy link
Contributor

This allows the configured root url to be changed and for subsequent
requests to reflect it.

@jacobstr jacobstr force-pushed the live-urls branch 2 times, most recently from 1e680e5 to 50aa3a5 Compare August 22, 2015 02:04
This allows the configured root url to be changed and for subsequent
requests to reflect said change in the root url.
@tschaub
Copy link
Member

tschaub commented Aug 23, 2015

This looks nice @jacobstr. We're using urls.WORKSPACES elsewhere, so that will need changing when upgrading.

Please merge.

@tschaub
Copy link
Member

tschaub commented Aug 23, 2015

Feel free to cut a release after merging. Since we're pre 1.0, this should be a minor release (this will keep ^0.7 dependents from getting upgrades). I'll put this in the readme, but the release procedure is basically this:

# on master
npm version minor && git push --tags origin master && npm publish

The postpublish script will update the docs.

jacobstr added a commit that referenced this pull request Aug 23, 2015
Dynamically generate urls from current config.
@jacobstr jacobstr merged commit 155bb97 into master Aug 23, 2015
@jacobstr jacobstr deleted the live-urls branch September 11, 2015 20:39
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