-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
add more modules to StdLabels
/ MoreLabels
#13110
Comments
Personally, I don't have see any problems with doing so. @gasche, @Octachron: do you see any issue with this proposed addition? |
For users that sounds fine. It opens a can of bikeshedding worms as we will have to pick new label names, but I think that we should survive this, and the result would be an improvement. Currently the workflow around I think that we did not think of doing this for a while because we collectively assumed that no one really cared about the *Labels modules -- we never get user feedback about them. I would certainly have guessed that label lovers have all migrated to use Core or something. It is useful feedback from @anmonteiro that there is interest in more of it for the standard library. (Personally I like judicious use of labels and wouldn't mind increased usage of labels in the default modules themselves; for example I rarely remember how to use |
For the record, I personally do care about the labelled modules. I also recall a few posts on Discuss of people saying they like to use them. If there is no mass feedback about them, maybe it’s because it’s not something that comes to mind when answering to the free-form comments of the OCaml survey? Or maybe they are indeed underused, which may be due to the fact that their existence is hardly publicized (I can’t think of any mention except the one line in the reference documentation of each module). |
As a historical comment, all modules were labelized in ocaml 3.00. |
I usually start new projects with
-open StdLabels
(orinclude StdLabels
, or similar).Recently, I noticed that e.g. the
Seq
module doesn't have a labelled version, even though most of its functions are good candidates (likeList
orArray
).Would it be a welcome addition to add more modules to either
StdLabels
,MoreLabels
or a new module?The text was updated successfully, but these errors were encountered: