Skip to content

Clarify CA1509 global configuration requirement - #55946

Merged
gewarren merged 2 commits into
mainfrom
copilot/fix-ca1509-configuration
Sep 11, 2026
Merged

Clarify CA1509 global configuration requirement#55946
gewarren merged 2 commits into
mainfrom
copilot/fix-ca1509-configuration

Conversation

Copilot AI commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

CA1509 doesn't activate when configured in .editorconfig, but the article didn't identify the required configuration scope.

  • Configuration guidance
    • Add a note that CA1509 must be enabled in .globalconfig.
    • Clarify that .editorconfig can't enable the rule.

Internal previews

File Preview link
docs/fundamentals/code-analysis/quality-rules/ca1509.md Learn preview

Build report

Copilot AI lite review requested due to automatic review settings September 10, 2026 23:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: gewarren <24882762+gewarren@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue with CA1509 configuration in .globalconfig Clarify CA1509 global configuration requirement Sep 10, 2026
Copilot AI requested a review from gewarren September 10, 2026 23:14
@gewarren
gewarren marked this pull request as ready for review September 10, 2026 23:17
@gewarren
gewarren requested a review from a team as a code owner September 10, 2026 23:17
@gewarren
gewarren enabled auto-merge (squash) September 10, 2026 23:18

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

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 136ce40 into main Sep 11, 2026
13 checks passed
@gewarren
gewarren deleted the copilot/fix-ca1509-configuration branch September 11, 2026 14:10
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.

Not obvious that CA1509 Invalid entry in code metrics configuration file needs to be enabled in .globalconfig

4 participants