Skip to content

Add note about internal representation of .NET runtime types #44672

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 4, 2025
Merged

Add note about internal representation of .NET runtime types #44672

merged 1 commit into from
Feb 4, 2025

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented Feb 4, 2025 •
edited by github-actions bot
Loading

Depending on internal representation is equivalent of private reflection that is not supported for types proviced by .NET runtime.

Internal previews

📄 File 🔗 Preview link
docs/standard/native-interop/best-practices.md Native interoperability best practices

@jkotas jkotas requested review from a team and jkoritzinsky as code owners February 4, 2025 03:49
@dotnetrepoman dotnetrepoman bot added this to the February 2025 milestone Feb 4, 2025
Copilot AI reviewed Feb 4, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@jkotas jkotas mentioned this pull request Feb 4, 2025
Merged
BillWagner
BillWagner approved these changes Feb 4, 2025
View reviewed changes
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 @jkotas

This LGTM. I'll :shipit:

@BillWagner BillWagner merged commit d2cb2d1 into dotnet:main Feb 4, 2025
10 checks passed
@jkotas jkotas deleted the internal-layout branch July 27, 2025 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@BillWagner BillWagner BillWagner approved these changes

@jkoritzinsky jkoritzinsky jkoritzinsky approved these changes

@AaronRobinsonMSFT AaronRobinsonMSFT AaronRobinsonMSFT approved these changes

Assignees

No one assigned

Labels

dotnet-fundamentals/svc

Projects

None yet

Milestone

February 2025

Development

Successfully merging this pull request may close these issues.

4 participants

@jkotas @BillWagner @jkoritzinsky @AaronRobinsonMSFT