8000 Add more GHCs to CI config, fix hlint warnings and haddock links by jhrcek · Pull Request #41 · fumieval/extensible · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add more GHCs to CI config, fix hlint warnings and haddock links #41

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

Merged
merged 4 commits into from
Jun 26, 2025

Conversation

jhrcek
Copy link
Contributor
@jhrcek jhrcek commented Apr 11, 2025

Hello,
I like this package. I'm interested in using it in my project.
I'm just opening this "probing" PR to check whether you're actively maintaining this project and to see if you're open to accepting fixes (should I need some in the future, not that I do now 😄 )

@@ -10,22 +10,23 @@
--
-- * Basic
--
-- * 'Data.Extensible.Class': basic membership operations.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Explanation of these changes:
These links are currently not working as you might have intended. Try clicking any of them here https://hackage.haskell.org/package/extensible-0.9.2/docs/Data-Extensible.html and you'll see it takes you back to the same page.

The proper way to link to modules is to surround their names in double quotes as explained in haddock documentation https://haskell-haddock.readthedocs.io/latest/markup.html#linking-to-modules

@@ -96,7 +103,6 @@ library
, StateVar
, tagged
, template-haskell
, text
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing unused dependency.

@fumieval
Copy link
Owner

Thanks!

@fumieval fumieval merged commit 0ea3f19 into fumieval:master Jun 26, 2025
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
0