-
Notifications
You must be signed in to change notification settings - Fork 14
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
Re-export IsList class from a friendlier module #54
Comments
I've given this a bit more thought. And I've noticed that both Given this I'm refining my proposals to re-exporting |
I'm quite fond of the |
If it's to be a more libraryish thing, I think it would be nice to add more sequence operations. |
Do you mean as class methods? These would all have varying complexities for different container types. It would be a lesson for the user to choose the right data structure for the task at hand; as it is in many other languages. A common interface for "sequences" -- separately from things that are |
@cigsender, once you have |
Having the methods you listed (among others) as class methods would allow for efficient overrides where possible. Associated types such as |
No, they solve a different problem. Both problems are important.
|
The reason is that |
That sounds good to me. |
@TeofilC if you'd like to move this forward, please put up a draft MR. |
Dear CLC members, let's vote on https://gitlab.haskell.org/ghc/ghc/-/merge_requests/8124, which moves +1 from me. |
+1 |
Makes sense +1 |
+1 Seems a more appropriate place and it is re-exported from |
With 4 votes in favor out of 6 possible, the proposal is approved by CLC, thanks. |
I'm trying to summarise the state of this proposal as part of my volunteering effort to track the progress of all
Please, let me know if you find any mistakes 🙂 |
Hoogle tells me that the IsList class is only exported from
GHC.Exts
inbase
. I think this class is quite useful and it would be nice to make it available from a non-internal module.A bunch of prelude replacement packages seem to re-export it from more friendly modules so there seems to be a desire for this.
I'd recommend re-exporting
IsList
fromPrelude
orData.List
. I think that's the two modules where it makes most sense.The text was updated successfully, but these errors were encountered: