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

Use displayException instead of show #417

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

edsko
Copy link
Contributor

@edsko edsko commented Nov 6, 2024

This is particularly useful in ghc 9.10 and up, where displayException shows the attached annotations, including backtraces (if any).

@MaximilianAlgehed
Copy link
Collaborator

Thank you for this contribution! If you could fix the CI failures (in this case you probably just need to add some pragmas guarding against old compilers) I think this would be a valuable contribution.

@MaximilianAlgehed MaximilianAlgehed self-requested a review November 7, 2024 11:08
This is particularly useful in ghc 9.10 and up, where `displayException` shows
the attacked annotations, including backtraces (if any).
@edsko edsko force-pushed the edsko/displayException branch from e0612ba to e538a68 Compare November 8, 2024 12:15
@edsko
Copy link
Contributor Author

edsko commented Nov 8, 2024

Have pushed what's hopefully a fix. This is the first time in my Haskell career I have had CI fail due to a test against hugs, feels like a rite of passage somehow 😬

@MaximilianAlgehed
Copy link
Collaborator

Great! Any day now we will support UHC too...

@MaximilianAlgehed MaximilianAlgehed merged commit 580a54d into nick8325:master Nov 8, 2024
52 checks passed
@edsko edsko deleted the edsko/displayException branch November 8, 2024 12:35
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.

2 participants