Skip to content

Disallow edits on system.reflection.dispatchproxy#12350

Merged
gewarren merged 2 commits intodotnet:mainfrom
gewarren:disable-edits-dispatchproxy
Feb 26, 2026
Merged

Disallow edits on system.reflection.dispatchproxy#12350
gewarren merged 2 commits intodotnet:mainfrom
gewarren:disable-edits-dispatchproxy

Conversation

@gewarren
Copy link
Contributor

Follow up to dotnet/runtime#124361.

@gewarren gewarren requested a review from a team as a code owner February 25, 2026 22:27
Copilot AI review requested due to automatic review settings February 25, 2026 22:27
@gewarren gewarren enabled auto-merge (squash) February 25, 2026 22:27
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.

Pull request overview

This PR adds System.Reflection.DispatchProxy to the disallow-edits configuration, following up on PR #124361. The change aims to prevent direct edits to auto-generated API documentation for this namespace, requiring contributors to update the source code comments in the dotnet/runtime repository instead.

Changes:

  • Added System.Reflection.DispatchProxy to the open_to_public_contributors exclusion list in docfx.json
  • Added a file matching pattern for System.Reflection.DispatchProxy in the disallow-edits GitHub policy

Reviewed changes

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

File Description
docfx.json Added System.Reflection.DispatchProxy to the open_to_public_contributors section to prevent public contributions
.github/policies/disallow-edits.yml Added file pattern to automatically close PRs that attempt to edit System.Reflection.DispatchProxy documentation

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.

This LGTM. Let's :shipit:

@gewarren gewarren merged commit fe847c6 into dotnet:main Feb 26, 2026
7 checks passed
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.

3 participants