Skip to content

Move RA0056 check to the EntitySystemSubscriptionGeneratorErrorAnalyzer - #6938

Open
Winkarst-cpu wants to merge 3 commits into
space-wizards:masterfrom
Winkarst-cpu:fix-notparial-error-subscription-generator
Open

Move RA0056 check to the EntitySystemSubscriptionGeneratorErrorAnalyzer#6938
Winkarst-cpu wants to merge 3 commits into
space-wizards:masterfrom
Winkarst-cpu:fix-notparial-error-subscription-generator

Conversation

@Winkarst-cpu

@Winkarst-cpu Winkarst-cpu commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Fixes #6771.

Screenshot_20260807_171617

Breaking changes:
Removed PartialTypeInfo.CheckPartialDiagnostic since it was obsolete and unused.
Removed PartialTypeInfo.SyntaxLocation since it became unused.

@Tayrtahn

Tayrtahn commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Am I right in thinking that this removes the only use of PartialTypeInfo.Location? If so, it would probably be good to remove that field from PartialTypeInfo so instances can be compared without needing to remember to use the custom comparer (since forgetting to use it very costly!)

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.

RA0056 Analyzer does not work

2 participants