Skip to content

Conversation

@albert-du
Copy link
Contributor

Adds F# code snippets for System.ArraySegment

@opbld30
Copy link

opbld30 commented Dec 2, 2021

Docs Build status updates of commit 1e9ccca:

✅ Validation status: passed

File Status Preview URL Details
samples/snippets/fsharp/VS_Snippets_CLR_System/system.ArraySegment/FS/arraysegment.fs ✅Succeeded View
samples/snippets/fsharp/VS_Snippets_CLR_System/system.ArraySegment/FS/fs.fsproj ✅Succeeded
samples/snippets/fsharp/VS_Snippets_CLR_System/system.ArraySegment/FS/segmentexample.fs ✅Succeeded View
samples/snippets/fsharp/VS_Snippets_CLR_System/system.arraysegment.class/fs/example1.fs ✅Succeeded View
samples/snippets/fsharp/VS_Snippets_CLR_System/system.arraysegment.class/fs/fs.fsproj ✅Succeeded
xml/System/ArraySegment`1.xml ✅Succeeded View

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:

@albert-du albert-du marked this pull request as ready for review December 2, 2021 06:18
@albert-du albert-du requested a review from a team as a code owner December 2, 2021 06:18
@albert-du
Copy link
Contributor Author

@dsyme @baronfel Could you review?

@BillWagner BillWagner requested review from baronfel and dsyme December 2, 2021 15:48
Copy link
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Examples look good to me, thanks for your persistence on these!

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again @albert-du

I'll :shipit: now.

@BillWagner BillWagner merged commit 76b6db3 into dotnet:main Dec 3, 2021
@albert-du albert-du deleted the system-arraysegment-fsharp-snippets branch January 19, 2022 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants