Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

Checklist

  • I have read the Contribution Guide ;
  • There is an approved issue describing the change when contributing a new analyzer or suppressor ;
  • I have added tests that prove my fix is effective or that my feature works ;
  • I have added necessary documentation (if appropriate) ;

Short description of what this resolves:

UNT0034 and UNT0035 were miscategorized as Performance. They suggest implicit conversions between Vector2/Vector3, which offer no performance benefit—the implicit operator calls the same constructor. These are readability improvements.

Changes proposed in this pull request:

  • Add DiagnosticCategory.Readability field to support readability-focused diagnostics
  • Recategorize UNT0034 (Vector3→Vector2 conversion) from Performance to Readability
  • Recategorize UNT0035 (Vector2→Vector3 conversion) from Performance to Readability
  • Update diagnostic index documentation to reflect new categories

💡 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.

Copilot AI and others added 2 commits January 20, 2026 14:09
- Add new DiagnosticCategory.Readability field
- Update UNT0034 (Vector3Conversion) to use Readability category
- Update UNT0035 (Vector2Conversion) to use Readability category
- Fixes #409

Co-authored-by: sailro <638167+sailro@users.noreply.github.com>
…UNT0035

Co-authored-by: sailro <638167+sailro@users.noreply.github.com>
Copilot AI changed the title [WIP] Introduce DiagnosticCategory.Readability field for analyzers Add Readability category for vector conversion diagnostics Jan 20, 2026
Copilot AI requested a review from sailro January 20, 2026 14:15
@sailro sailro marked this pull request as ready for review January 20, 2026 15:16
@sailro sailro requested a review from a team as a code owner January 20, 2026 15:16
@sailro sailro merged commit d9cefe7 into main Jan 20, 2026
6 checks passed
@sailro sailro deleted the copilot/add-readability-diagnostic-category branch January 20, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants