Skip to content
This repository has been archived by the owner on Nov 26, 2024. It is now read-only.

Live -> master #313

Merged
merged 30 commits into from
Jun 20, 2018
Merged

Live -> master #313

merged 30 commits into from
Jun 20, 2018

Conversation

cartermp
Copy link
Contributor

No description provided.

j2ghz and others added 29 commits December 7, 2016 08:12
fixed signature for copy function
Running the following in F# Interactive will help explain why I believe this change is necessary:
> List.foldBack (fun elem acc -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 4 elem: 3 elem: 2 elem: 1 val it : int = 10
> List.foldBack (fun acc elem -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 0 elem: 4 elem: 7 elem: 9 val it : int = 10

The accumulator is the second argument in foldBack (contrary to fold where it is the first).
Positioning the arguments in reverse in an example is a great way to confuse a reader.

This also helps promote the fact that the state and list arguments to the folding function
are in the same order as their lambda counterparts.

Feel free to disagree :-).
atan2(y, x) is related to atan(y/x), *not* atan(x/y)
Added missing line break to class definition of Microsoft.FSharp.Reflection.UnionCaseInfo
Added missing line break
Fix typo in Array2D.init docs
Removed Duplicate Members in CustomOperationAttribute. Issue #266
Document that Map.pick can throw KeynotFoundException
Update async.startchild['t]-method-[fsharp].md
…able (#301)

The `string` function casts to `IFormattable` internally. The original text only mentioned integer and floating point values, but in fact, there are many more values in the BCL that implement IFormattable. I improved the text to be more precise.

Furthermore, special treatment of `null` was not mentioned, I've added that. Expanded supported platforms, and fixed a typo.
The except function added in fsharp/fslang-design#41 doesn't have a doc here.
@dend
Copy link
Contributor

dend commented Jun 20, 2018

❌ Validation status: errors

File Status Preview URL Details
❌Error Details

  • [Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/visualfsharpdocs, working directory: W:\eh0a-s, target branch: master, target branch commit: 381ffd6, source branch: live, source branch commit: .

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@dend
Copy link
Contributor

dend commented Jun 20, 2018

❌ Validation status: errors

File Status Preview URL Details
❌Error Details

  • [Error] Error occurred: Microsoft.OpenPublishing.Build.DataAccessor.Interface.GitAccessException: Error occurred when accessing GitHub. Details: (BadGateway) Server Error
    at Microsoft.OpenPublishing.Build.DataAccessor.Git.GitHubAccessor.<GenerateExceptionAsync>d__77.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult() at Microsoft.OpenPublishing.Build.DataAccessor.Git.BaseGitRepositoryAccessor.&lt;RunRequestAsync&gt;d__1341.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.OpenPublishing.Build.DataAccessor.Git.GitHubAccessor.<GetCommitCoreAsync>d__38.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.OpenPublishing.Build.Applications.GitContributorInformationResolver.GitInformationResolver.<UpdateGitHubUserProfileAsync>d__12.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.OpenPublishing.Build.Applications.GitContributorInformationResolver.GitInformationResolver.<ResolveContributorInfoAsync>d__9.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
    at Microsoft.OpenPublishing.Build.Applications.GitContributorInformationResolver.GitContributorInformationResolver.<ResolveGitUserProfileAsync>d__56.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.OpenPublishing.Build.Applications.GitContributorInformationResolver.GitContributorInformationResolver.<ResolveGitContributorInformationAsync>d__58.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.OpenPublishing.Build.Applications.GitContributorInformationResolver.Program.<RunAsync>d__2.MoveNext()
  • [Error] System.AggregateException: One or more errors occurred. ---> System.ArgumentNullException: Value cannot be null.
    Parameter name: historyInformation.PublishCommitsHistoryFilePath
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.GitRepositoryInformationFileProvider..ctor(Uri repositoryUrl, String commitId, GitRepositoryAccessCredential accessCredential, ResolveUserProfileParameter resolveUserProfileParameter, CommitsHistoryInformation historyInformation, IGitRepositoryAccessorFactory gitRepositoryAccessorFactory)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.GitContributorInformationResolverFactory.<GetResolverAsync>d__3.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.CachedGitContributorInformationResolverFactory.<GetResolverAsync>d__4.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<GetGitContributorInformationAsync>d__22.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<>c__DisplayClass19_0.<<BuildNonReferenceAsync>b__0>d.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<BuildNonReferenceAsync>d__19.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetConceptualDocumentGitContributorInformation.<BuildCoreAsync>d__5.MoveNext()
    --- End of inner exception stack trace ---
    at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification) at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.CachedGitContributorInformationResolverFactory.Dispose() at Microsoft.DocAsCode.Build.Common.DisposableDocumentProcessor.Dispose() at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.Dispose() at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.BuildCore(DocumentBuildParameters parameter, IMarkdownServiceProvider markdownServiceProvider, BuildInfo currentBuildInfo, BuildInfo lastBuildInfo) at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.Build(IList1 parameters, String outputDirectory)
    at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument(BuildJsonConfig config, TemplateManager templateManager, String baseDirectory, String outputDirectory, String pluginDirectory, String templateDirectory)
    at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument()
    ---> (Inner Exception #0) System.ArgumentNullException: Value cannot be null.
    Parameter name: historyInformation.PublishCommitsHistoryFilePath
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.GitRepositoryInformationFileProvider..ctor(Uri repositoryUrl, String commitId, GitRepositoryAccessCredential accessCredential, ResolveUserProfileParameter resolveUserProfileParameter, CommitsHistoryInformation historyInformation, IGitRepositoryAccessorFactory gitRepositoryAccessorFactory)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.GitContributorInformationResolverFactory.<GetResolverAsync>d__3.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.CachedGitContributorInformationResolverFactory.<GetResolverAsync>d__4.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<GetGitContributorInformationAsync>d__22.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<>c__DisplayClass19_0.<<BuildNonReferenceAsync>b__0>d.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetGitContributorInformation.<BuildNonReferenceAsync>d__19.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at Microsoft.DocAsCode.EntityModel.Plugins.OpenPublishing.SetConceptualDocumentGitContributorInformation.<BuildCoreAsync>d__5.MoveNext()<---

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@cartermp cartermp closed this Jun 20, 2018
@cartermp cartermp reopened this Jun 20, 2018
@dend
Copy link
Contributor

dend commented Jun 20, 2018

⚠️ Validation status: warnings

File Status Preview URL Details
docs/conceptual/collections.array-module-[fsharp].md ⚠️Warning Details
docs/conceptual/collections.list-module-[fsharp].md ⚠️Warning Details
CONTRIBUTING.md ✅Succeeded
docs/conceptual/array.maxby['t,'u]-function-[fsharp].md ✅Succeeded
docs/conceptual/array2d.init['t]-function-[fsharp].md ✅Succeeded
docs/conceptual/async.startchild['t]-method-[fsharp].md ✅Succeeded
docs/conceptual/core.customoperationattribute-class-[fsharp].md ✅Succeeded
docs/conceptual/core.operators-module-[fsharp].md ✅Succeeded
docs/conceptual/core.printf-module-[fsharp].md ✅Succeeded
docs/conceptual/list.exists2['t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.fold2['t1,'t2,'state]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.foldback2['t1,'t2,'state]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.forall2['t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.iter2['t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.iteri2['t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.map2['t1,'t2,'u]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.map3['t1,'t2,'t3,'u]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.mapi2['t1,'t2,'u]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.zip3['t1,'t2,'t3]-function-[fsharp].md ✅Succeeded
docs/conceptual/list.zip['t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/map.pick['key,'t,'u]-function-[fsharp].md ✅Succeeded
docs/conceptual/operators.atan2[^t1,'t2]-function-[fsharp].md ✅Succeeded
docs/conceptual/operators.string[^t]-function-[fsharp].md ✅Succeeded
docs/conceptual/quotations.patterns-module-[fsharp].md ✅Succeeded
docs/conceptual/reflection.fsharptype-class-[fsharp].md ✅Succeeded
docs/conceptual/reflection.unioncaseinfo-class-[fsharp].md ✅Succeeded
docs/conceptual/runtimehelpers.leafexpressionconverter-module-[fsharp].md ✅Succeeded
docs/conceptual/seq.concat['collection,'t]-function-[fsharp].md ✅Succeeded
docs/conceptual/snippets/fslists/snippet41.fs ✅Succeeded
docs/conceptual/strings-[fsharp].md ✅Succeeded
docs/conceptual/walkthrough-using-visual-fsharp-to-create,-debug,-and-deploy-an-application.md ✅Succeeded

docs/conceptual/collections.array-module-[fsharp].md

docs/conceptual/collections.list-module-[fsharp].md

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@cartermp cartermp merged commit fa9f433 into master Jun 20, 2018
cartermp added a commit that referenced this pull request Jun 20, 2018
* Live -> master (#313)

* Fix broken table (#269)

* Fix code indentation (#267)

* corrected the output (#276)

* Update collections.array - fsharp .md (#271)

fixed signature for copy function

* fix and consolidate list.xyz2/xyz3 doc for input lists with different lengths (#282)

* Fix escape sequence. (#285)

* Add missing space (#284)

* Fix order of arguments in foldBack lambda (#286)

Running the following in F# Interactive will help explain why I believe this change is necessary:
> List.foldBack (fun elem acc -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 4 elem: 3 elem: 2 elem: 1 val it : int = 10
> List.foldBack (fun acc elem -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 0 elem: 4 elem: 7 elem: 9 val it : int = 10

The accumulator is the second argument in foldBack (contrary to fold where it is the first).
Positioning the arguments in reverse in an example is a great way to confuse a reader.

This also helps promote the fact that the state and list arguments to the folding function
are in the same order as their lambda counterparts.

Feel free to disagree :-).

* Fix Operators.atan2 description (#287)

atan2(y, x) is related to atan(y/x), *not* atan(x/y)

* Fixed code formatting (#288)

* Corrected prefix minus to prefix plus (#289)

* Added missing line break

Added missing line break to class definition of Microsoft.FSharp.Reflection.UnionCaseInfo

* Removed Missing CustomOperationAttribute Issue #266

* JoinConditionWord was Accidentally removed despite not being duplicated. Fixing this.

* Fix typo in Array2D.init docs

* Document that Map.pick can throw KeynotFoundException

* Update async.startchild['t]-method-[fsharp].md

Specify timeout behaviour

* Fix List.compareWith case

* Update walkthrough-using-visual-fsharp-to-create,-debug,-and-deploy-an-application.md

* Added treatment of null-values, and fixed incomplete text on IFormattable (#301)

The `string` function casts to `IFormattable` internally. The original text only mentioned integer and floating point values, but in fact, there are many more values in the BCL that implement IFormattable. I improved the text to be more precise.

Furthermore, special treatment of `null` was not mentioned, I've added that. Expanded supported platforms, and fixed a typo.

* Update quotations.patterns-module-[fsharp].md (#302)

Thanks @Columpio

* Clarified how ties are resolved (#304)

* Add except to list modules (#305)

The except function added in fsharp/fslang-design#41 doesn't have a doc here.

* Correcting the Return Value in list.partition['t]-function-[fsharp].md  (#309)

* Fix broken table (#269)

* Fix code indentation (#267)

* corrected the output (#276)

* Update collections.array - fsharp .md (#271)

fixed signature for copy function

* fix and consolidate list.xyz2/xyz3 doc for input lists with different lengths (#282)

* Fix escape sequence. (#285)

* Add missing space (#284)

* Fix order of arguments in foldBack lambda (#286)

Running the following in F# Interactive will help explain why I believe this change is necessary:
> List.foldBack (fun elem acc -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 4 elem: 3 elem: 2 elem: 1 val it : int = 10
> List.foldBack (fun acc elem -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 0 elem: 4 elem: 7 elem: 9 val it : int = 10

The accumulator is the second argument in foldBack (contrary to fold where it is the first).
Positioning the arguments in reverse in an example is a great way to confuse a reader.

This also helps promote the fact that the state and list arguments to the folding function
are in the same order as their lambda counterparts.

Feel free to disagree :-).

* Fix Operators.atan2 description (#287)

atan2(y, x) is related to atan(y/x), *not* atan(x/y)

* Fixed code formatting (#288)

* Corrected prefix minus to prefix plus (#289)

* Added missing line break

Added missing line break to class definition of Microsoft.FSharp.Reflection.UnionCaseInfo

* Removed Missing CustomOperationAttribute Issue #266

* JoinConditionWord was Accidentally removed despite not being duplicated. Fixing this.

* Fix typo in Array2D.init docs

* Document that Map.pick can throw KeynotFoundException

* Update async.startchild['t]-method-[fsharp].md

Specify timeout behaviour

* Update walkthrough-using-visual-fsharp-to-create,-debug,-and-deploy-an-application.md

* Added treatment of null-values, and fixed incomplete text on IFormattable (#301)

The `string` function casts to `IFormattable` internally. The original text only mentioned integer and floating point values, but in fact, there are many more values in the BCL that implement IFormattable. I improved the text to be more precise.

Furthermore, special treatment of `null` was not mentioned, I've added that. Expanded supported platforms, and fixed a typo.

* Update list.partition['t]-function-[fsharp].md

* Fixed formatting of README (#291)
@cartermp cartermp mentioned this pull request Jun 20, 2018
cartermp added a commit that referenced this pull request Jun 20, 2018
* Live -> master (#313)

* Fix broken table (#269)

* Fix code indentation (#267)

* corrected the output (#276)

* Update collections.array - fsharp .md (#271)

fixed signature for copy function

* fix and consolidate list.xyz2/xyz3 doc for input lists with different lengths (#282)

* Fix escape sequence. (#285)

* Add missing space (#284)

* Fix order of arguments in foldBack lambda (#286)

Running the following in F# Interactive will help explain why I believe this change is necessary:
> List.foldBack (fun elem acc -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 4 elem: 3 elem: 2 elem: 1 val it : int = 10
> List.foldBack (fun acc elem -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 0 elem: 4 elem: 7 elem: 9 val it : int = 10

The accumulator is the second argument in foldBack (contrary to fold where it is the first).
Positioning the arguments in reverse in an example is a great way to confuse a reader.

This also helps promote the fact that the state and list arguments to the folding function
are in the same order as their lambda counterparts.

Feel free to disagree :-).

* Fix Operators.atan2 description (#287)

atan2(y, x) is related to atan(y/x), *not* atan(x/y)

* Fixed code formatting (#288)

* Corrected prefix minus to prefix plus (#289)

* Added missing line break

Added missing line break to class definition of Microsoft.FSharp.Reflection.UnionCaseInfo

* Removed Missing CustomOperationAttribute Issue #266

* JoinConditionWord was Accidentally removed despite not being duplicated. Fixing this.

* Fix typo in Array2D.init docs

* Document that Map.pick can throw KeynotFoundException

* Update async.startchild['t]-method-[fsharp].md

Specify timeout behaviour

* Fix List.compareWith case

* Update walkthrough-using-visual-fsharp-to-create,-debug,-and-deploy-an-application.md

* Added treatment of null-values, and fixed incomplete text on IFormattable (#301)

The `string` function casts to `IFormattable` internally. The original text only mentioned integer and floating point values, but in fact, there are many more values in the BCL that implement IFormattable. I improved the text to be more precise.

Furthermore, special treatment of `null` was not mentioned, I've added that. Expanded supported platforms, and fixed a typo.

* Update quotations.patterns-module-[fsharp].md (#302)

Thanks @Columpio

* Clarified how ties are resolved (#304)

* Add except to list modules (#305)

The except function added in fsharp/fslang-design#41 doesn't have a doc here.

* Correcting the Return Value in list.partition['t]-function-[fsharp].md  (#309)

* Fix broken table (#269)

* Fix code indentation (#267)

* corrected the output (#276)

* Update collections.array - fsharp .md (#271)

fixed signature for copy function

* fix and consolidate list.xyz2/xyz3 doc for input lists with different lengths (#282)

* Fix escape sequence. (#285)

* Add missing space (#284)

* Fix order of arguments in foldBack lambda (#286)

Running the following in F# Interactive will help explain why I believe this change is necessary:
> List.foldBack (fun elem acc -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 4 elem: 3 elem: 2 elem: 1 val it : int = 10
> List.foldBack (fun acc elem -> printf "elem: %A " elem; elem + acc) [1;2;3;4] 0;;
elem: 0 elem: 4 elem: 7 elem: 9 val it : int = 10

The accumulator is the second argument in foldBack (contrary to fold where it is the first).
Positioning the arguments in reverse in an example is a great way to confuse a reader.

This also helps promote the fact that the state and list arguments to the folding function
are in the same order as their lambda counterparts.

Feel free to disagree :-).

* Fix Operators.atan2 description (#287)

atan2(y, x) is related to atan(y/x), *not* atan(x/y)

* Fixed code formatting (#288)

* Corrected prefix minus to prefix plus (#289)

* Added missing line break

Added missing line break to class definition of Microsoft.FSharp.Reflection.UnionCaseInfo

* Removed Missing CustomOperationAttribute Issue #266

* JoinConditionWord was Accidentally removed despite not being duplicated. Fixing this.

* Fix typo in Array2D.init docs

* Document that Map.pick can throw KeynotFoundException

* Update async.startchild['t]-method-[fsharp].md

Specify timeout behaviour

* Update walkthrough-using-visual-fsharp-to-create,-debug,-and-deploy-an-application.md

* Added treatment of null-values, and fixed incomplete text on IFormattable (#301)

The `string` function casts to `IFormattable` internally. The original text only mentioned integer and floating point values, but in fact, there are many more values in the BCL that implement IFormattable. I improved the text to be more precise.

Furthermore, special treatment of `null` was not mentioned, I've added that. Expanded supported platforms, and fixed a typo.

* Update list.partition['t]-function-[fsharp].md

* Fixed formatting of README (#291)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.