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

Bad escaping of lambdas in Haddocks #315

Closed
georgefst opened this issue Mar 13, 2024 · 3 comments · Fixed by #318
Closed

Bad escaping of lambdas in Haddocks #315

georgefst opened this issue Mar 13, 2024 · 3 comments · Fixed by #318

Comments

@georgefst
Copy link

image
@abigailalice
Copy link
Contributor

The stuff on construction also has examples of this, showing value applications when they should have type applications

> :t buildADT Task "Pending"
buildADT Task "Pending" :: ADT Task Expr
> :t buildADT Task "Complete"
buildADT Task "Complete" :: CompletedTask Expr -> ADT Task Expr

abigailalice added a commit to abigailalice/rel8 that referenced this issue Mar 14, 2024
ocharles added a commit that referenced this issue Mar 19, 2024
* Fix some unescaped markup in haddocks

This addresses #315

* Expand documentation on unsafeDefault

This copies the documentation in the cookbook to the haddocks,
addressing #233.

* Update docs/concepts/insert.rst

Co-authored-by: Ollie Charles <ollie@ocharles.org.uk>

---------

Co-authored-by: Ollie Charles <ollie@ocharles.org.uk>
@ocharles
Copy link
Contributor

@abigailalice Am I right in thinking you fixed this?

@abigailalice
Copy link
Contributor

I found at least one more example, I'm going through things thoroughly to see if there are any more.

abigailalice added a commit to abigailalice/rel8 that referenced this issue Mar 22, 2024
ocharles pushed a commit that referenced this issue Apr 3, 2024
abigailalice added a commit to abigailalice/rel8 that referenced this issue Aug 3, 2024
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 a pull request may close this issue.

3 participants