-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add breaking change documentation for DAMT.All replacement in System.Reflection APIs #48922
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
Add breaking change documentation for DAMT.All replacement in System.Reflection APIs #48922
Conversation
…ion APIs Co-authored-by: gewarren <[email protected]>
Co-authored-by: gewarren <[email protected]>
Co-authored-by: gewarren <[email protected]>
Co-authored-by: gewarren <[email protected]>
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.
Pull Request Overview
This PR adds comprehensive breaking change documentation for .NET 10, documenting the replacement of DynamicallyAccessedMemberTypes.All with more restricted annotations in System.Reflection APIs. The change affects developers implementing IReflect or deriving from TypeInfo classes.
Key changes include:
- Creation of complete breaking change documentation explaining the annotation restrictions
- Addition of code examples showing required implementation updates
- Integration into the .NET 10 breaking changes documentation hierarchy
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/core/compatibility/toc.yml | Added new Reflection section entry for .NET 10 breaking changes |
| docs/core/compatibility/reflection/10.0/ireflect-damt-annotations.md | Complete breaking change article with examples and guidance |
| docs/core/compatibility/10.0.md | Added Reflection section to .NET 10 breaking changes overview |
|
(belated) LGTM |
Summary
This PR adds comprehensive breaking change documentation for .NET 10, documenting the replacement of
DynamicallyAccessedMemberTypes.Allwith more restricted annotations in System.Reflection APIs.Issue
Closes #48898
Changes
Documentation Added
Created breaking change documentation describing how
System.Reflection.IReflect.InvokeMember,System.Type.FindMembers, andSystem.Reflection.TypeInfo.DeclaredMembersAPIs have been updated to use more restricted annotations instead ofDynamicallyAccessedMemberTypes.Allin .NET 10 Preview 1.Files Created
docs/core/compatibility/reflection/10.0/ireflect-damt-annotations.mdIReflector deriving fromTypeInfoCode Snippets
docs/core/compatibility/reflection/10.0/snippets/ireflect-damt-annotations/csharp/MyType.cs- C# example showing required annotationsdocs/core/compatibility/reflection/10.0/snippets/ireflect-damt-annotations/vb/MyType.vb- Visual Basic equivalentFiles Modified
docs/core/compatibility/10.0.md- Added Reflection section to .NET 10 breaking changes overviewdocs/core/compatibility/toc.yml- Added new Reflection entry for .NET 10Example
The documentation includes code examples showing how to implement the required annotations when implementing
IReflect:Documentation Guidelines
ai-usage: ai-generated)Testing
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
Internal previews