Skip to content

Conversation

@piscisaureus
Copy link
Contributor

@piscisaureus
Copy link
ContributorAuthor

Proudly made with the github™ editor

@bnoordhuis
Copy link
Member

You should remove the R=@bnoordhuis from the commit log and maybe mention the revert commit.

@piscisaureus
Copy link
ContributorAuthor

Dang! now I still don't get to use the merge button.

`url.format()` doesn't take a `path` option; the change that introduced it was reverted in 913addb. PR: #546 Reviewed-by: Ben Noordhuis <[email protected]>
@piscisaureuspiscisaureus merged commit 60402b9 into v1.xJan 22, 2015
@piscisaureuspiscisaureus deleted the piscisaureus-patch-1 branch January 22, 2015 01:21
Sign up for freeto 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.

3 participants

@piscisaureus@bnoordhuis