-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
Version 3.0.0 → 4.0.0 #287
Conversation
Note that there are some in-flight pull requests that I will wait for before cutting this release. Specifically: |
``` | ||
|
||
```haskell | ||
https://example.com/foo/"bar?baz"?qux |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How to escape "
in this case? Just use the percent-encoding for that I assume. Can you use percent-encoding inside of quoted strings?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Profpatsch: It's not possible to use "
within a quoted path yet. If you want to support that then we'll have to standardize it
Quoting a percent-encoded path will not work because it will percent-encode the %
symbol when resolving the URL 🙂
This fixes the root cause of e24b3d9 so that we no longer need the workaround of using `Pretty.Text.renderIO`
No description provided.