Skip to content

Port System.Formats.Nrbf new docs #10168

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

Merged
merged 16 commits into from
Jul 30, 2024

Conversation

directhex
Copy link
Contributor

@dotnet/area-system-formats-nrbf @carlossanlop @gewarren PTAL

@directhex directhex requested a review from a team as a code owner July 26, 2024 20:40
@ghost ghost added the area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable. label Jul 26, 2024
Copy link

Learn Build status updates of commit eb28f1a:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Formats.Nrbf/ArrayRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/ClassRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/NrbfDecoder.xml ✅Succeeded View
xml/System.Formats.Nrbf/PayloadOptions.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord`1.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordId.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordType.xml ✅Succeeded View
xml/System.Formats.Nrbf/SZArrayRecord`1.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

I've found some issues, but they are not caused by the porting tool.

I've sent dotnet/runtime#105632 to fix the source of the truth and opened #10177 to address one issue once this PR gets resolved.

Please apply my suggestions before merging.

@directhex thank you!

directhex and others added 15 commits July 30, 2024 12:35
Copy link

Learn Build status updates of commit 6be3643:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Formats.Nrbf/ArrayRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/ClassRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/NrbfDecoder.xml ✅Succeeded View
xml/System.Formats.Nrbf/PayloadOptions.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord`1.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordId.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordType.xml ✅Succeeded View
xml/System.Formats.Nrbf/SZArrayRecord`1.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 38710ec:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Formats.Nrbf/ArrayRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/ClassRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/NrbfDecoder.xml ✅Succeeded View
xml/System.Formats.Nrbf/PayloadOptions.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/PrimitiveTypeRecord`1.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecord.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordId.xml ✅Succeeded View
xml/System.Formats.Nrbf/SerializationRecordType.xml ✅Succeeded View
xml/System.Formats.Nrbf/SZArrayRecord`1.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@carlossanlop carlossanlop merged commit 563e753 into dotnet:main Jul 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants