-
Notifications
You must be signed in to change notification settings - Fork 111
New functions added to List, Array, Seq modules in F# 4.0 are missing Docs #184
Comments
Thanks for putting this table together @liboz! |
@cartermp I plan on making at least some of the docs that are missing. Should I make multiple pull requests (something like one for each group of functions, i.e. List.contains, Array.contains, Seq.contains) or one big pull request or is there some other method that would be preferred? |
@liboz as soon as you have a new article, issue a PR. Unless you have 7 ready at once, one-by-one is fine and we'll make sure to review and merge them properly. |
Maybe we could combine with the works from @liboz :) |
The list at the top needs to be updated, now we have accepted PR from @ErikSchierboom
cc @cartermp |
Hi. Any progress? List.groupBy is missing documentation too at both https://docs.microsoft.com/en-us/dotnet/fsharp/language-reference/lists and https://msdn.microsoft.com/visualfsharpdocs/conceptual/collections.list-module-%5bfsharp%5d |
List.allPairs is also missing documentation #310 |
I will try to help with this, but it seems to me that the people who added the new functions should add the documentation too. They would know best exactly how the function works and how to use it, etc. |
Looks like most or even all these functions are now documented at |
@hickford That is only the bare minimum documentation generated from XML comments. There are no examples, for instance. That is inadequate. |
Many new functions were added in F# 4.0: https://github.com/fsharp/FSharpLangDesign/blob/master/FSharp-4.0/ListSeqArrayAdditions.md. Most of them do not have documentation yet.
Here's the list of functions that still need docs:
Total Remaining: 94
The text was updated successfully, but these errors were encountered: