Skip to content

chore: remove SyntaxFactory to improve performance and maintainability#103

Merged
glennawatson merged 2 commits intomainfrom
CP_UpdateOAPHFromObservable
Nov 4, 2024
Merged

chore: remove SyntaxFactory to improve performance and maintainability#103
glennawatson merged 2 commits intomainfrom
CP_UpdateOAPHFromObservable

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
@glennawatson glennawatson deleted the CP_UpdateOAPHFromObservable branch November 4, 2024 00:38
@github-actions
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