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 loader in CodeGeneratorEngineFactory #512

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

janvernieuwe
Copy link
Collaborator

The code update removes the dependency on Psr18ClientDiscovery and Psr18Loader and replaces it with StreamWrapperLoader in the CodeGeneratorEngineFactory. Additionally, a corresponding test for this change has been created in CodeGeneratorEngineFactoryTest to verify that the engine can be correctly loaded from the filesystem.

Fixes #510

@janvernieuwe janvernieuwe requested a review from veewee April 26, 2024 13:28
@janvernieuwe janvernieuwe self-assigned this Apr 26, 2024
The code update removes the dependency on Psr18ClientDiscovery and Psr18Loader and replaces it with StreamWrapperLoader in the CodeGeneratorEngineFactory. Additionally, a corresponding test for this change has been created in CodeGeneratorEngineFactoryTest to verify that the engine can be correctly loaded from the filesystem.

Fixes #510
@janvernieuwe janvernieuwe force-pushed the stream-wrapper-loader-as-default branch from 29be0c6 to 5209feb Compare April 26, 2024 13:31
@veewee veewee merged commit 05bded4 into v3.x Apr 26, 2024
24 checks passed
@janvernieuwe janvernieuwe deleted the stream-wrapper-loader-as-default branch April 26, 2024 13:48
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.

Providing a path to file for wizard gives invalid scheme error
2 participants