/utils

utils: add RelativeURL function #191

Merged
merged 1 commit into from Jan 21, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@rogpeppe @frankban @bz2 @mhilton @jujubot
@rogpeppe
Owner

rogpeppe commented Jan 21, 2016

This is the inverse of URL.ResolveReference, useful for calculating
a relative URL when needed.

(Review request: http://reviews.vapour.ws/r/3591/)
@frankban
Member

frankban commented Jan 21, 2016

👍 Thank you!
@bz2
Contributor

bz2 commented Jan 21, 2016

LGTM. I'd consider documenting or asserting that the input paths are normalised.
@mhilton
Member

mhilton commented Jan 21, 2016

👍
@rogpeppe
Owner

rogpeppe commented Jan 21, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Jan 21, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-utils

jujubot added a commit that referenced this pull request Jan 21, 2016

@jujubot
Merge pull request #191 from rogpeppe/016-relativeurl 
utils: add RelativeURL function

This is the inverse of URL.ResolveReference, useful for calculating
a relative URL when needed.

(Review request: http://reviews.vapour.ws/r/3591/)
fc8c377

@jujubot jujubot merged commit fc8c377 into juju:master Jan 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment