Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Add separate CodeFixes Analyzer #101

Merged
merged 8 commits into from
Nov 3, 2024
Merged

Conversation

ChrisPulman
Copy link
Member

@ChrisPulman ChrisPulman commented Oct 31, 2024

What kind of change does this PR introduce?

Update

What is the current behavior?

Fix for #99

What is the new behavior?

Separate CodeFixes from SourceGenerators
Update Windows Forms Generators for performance

What might this PR break?

None expected, now using two separate Analysers

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:

Separate CodeFixes from SourceGenerators
Fix for #99
Recommendation not to use Compile Analysers with Workspace Analysers - moved Generator analytics back to SourceGenerator Project
Ensure that we pass on Attributes placed on Fields, handle Property Declaration Attributes on Fields
@ChrisPulman ChrisPulman merged commit 3467bf4 into main Nov 3, 2024
1 check passed
@ChrisPulman ChrisPulman deleted the CP_SeperateAnalysers branch November 3, 2024 03:36
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants