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

escape space as +, not %20 #8

Merged
merged 1 commit into from Sep 6, 2012
Merged

escape space as +, not %20 #8

merged 1 commit into from Sep 6, 2012

Conversation

moritz
Copy link
Contributor

@moritz moritz commented Sep 4, 2012

this unbreaks fragment escaping for doc.perl6.org

this unbreaks fragment escaping for doc.perl6.org
@ronaldxs
Copy link
Collaborator

ronaldxs commented Sep 6, 2012

Hi,

I expect to merge this in the next day or two. The reason I won't add it right away is because I would like the change to come with at least two relevant tests (escape and unescape). Since I am going to be making changes myself soon I can add some tests if you like and currently plan to do so.

Thanks and look forward to merging,
Ron

ronaldxs added a commit that referenced this pull request Sep 6, 2012
escape space as +, not %20
@ronaldxs ronaldxs merged commit 71a5abe into ilyabelikin:master Sep 6, 2012
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.

None yet

2 participants