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

chore: remove SyntaxFactory to improve performance and maintainability #103

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

ChrisPulman
Copy link
Member

@ChrisPulman ChrisPulman commented Nov 4, 2024

What kind of change does this PR introduce?

Update

What is the current behavior?

Using SyntaxFactory to generate code, resulting in poor maintainability.

What is the new behavior?

Removed SyntaxFactory to improve performance and increase maintainability.

What might this PR break?

None expected; it should produce the same base code output from the generator.

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

This PR involves renaming and updating files to improve code structure and maintainability. The changes are expected to enhance the performance of the source generator without affecting the base code output.

For more details, you can view the files changed.

@glennawatson glennawatson changed the title Update OAPH From Observable chore: remove SyntaxFactory to improve performance and maintainability Nov 4, 2024
@glennawatson glennawatson merged commit fb8b47e into main Nov 4, 2024
1 check passed
@glennawatson glennawatson deleted the CP_UpdateOAPHFromObservable branch November 4, 2024 00:38
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