-
Notifications
You must be signed in to change notification settings - Fork 1.6k
System.Action<T1,T2> F# Snippets #7403
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
System.Action<T1,T2> F# Snippets #7403
Conversation
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
Docs Build status updates of commit e514800: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments
Marking as approved but please adjust
samples/snippets/fsharp/VS_Snippets_CLR_System/system.Action~2/fs/Action2.fs
Outdated
Show resolved
Hide resolved
with _ -> printfn "File write operation failed..." | ||
|
||
let concat = | ||
ConcatStrings( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly here
samples/snippets/fsharp/VS_Snippets_CLR_System/system.Action~2/fs/Delegate.fs
Outdated
Show resolved
Hide resolved
samples/snippets/fsharp/VS_Snippets_CLR_System/system.Action~2/fs/Action2.fs
Outdated
Show resolved
Hide resolved
Co-Authored-By: Don Syme <[email protected]>
👍 |
Docs Build status updates of commit c34a82b: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
@BillWagner this should be ready too |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM @albert-du
I'll now.
Summary
Adds F# code snippets for System.Action<T1,T2>. Revises the article to mention F# and link its lambda expression article.