Skip to content

Disallow edits on fileversioninfo and customreflectioncontext#12505

Merged
gewarren merged 2 commits intodotnet:mainfrom
gewarren:no-edits-april
Apr 10, 2026
Merged

Disallow edits on fileversioninfo and customreflectioncontext#12505
gewarren merged 2 commits intodotnet:mainfrom
gewarren:no-edits-april

Conversation

@gewarren
Copy link
Copy Markdown
Contributor

@gewarren gewarren commented Apr 9, 2026

@gewarren gewarren requested a review from a team as a code owner April 9, 2026 20:31
Copilot AI review requested due to automatic review settings April 9, 2026 20:31
Copy link
Copy Markdown
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 updates repo configuration to prevent public contributions to documentation that is generated from source-of-truth repos, specifically targeting System.Diagnostics.FileVersionInfo and System.Reflection.Context.CustomReflectionContext as a follow-up to upstream runtime changes.

Changes:

  • Update docfx.json open_to_public_contributors settings to mark additional generated reference YAML as not open to public contributors.
  • Update the GitHub policy .github/policies/disallow-edits.yml to close PRs that edit the corresponding generated XML inputs, including FileVersionInfo.xml and System.Reflection.Context/*.
  • Broaden the System.Composition disallow pattern to cover sub-namespaces.

Reviewed changes

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

File Description
docfx.json Adjusts open_to_public_contributors globs to restrict generated API YAML (but introduces glob issues with flattened output).
.github/policies/disallow-edits.yml Updates PR-closing policy patterns to disallow edits for FileVersionInfo and System.Reflection.Context XML.

Comment thread docfx.json Outdated
Comment thread docfx.json Outdated
Copy link
Copy Markdown
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 7ea4488 into dotnet:main Apr 10, 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