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

Added feature to suppress the generation of the output of client classes and interfaces. #4571

Merged

Conversation

paulomorgado
Copy link
Contributor

@paulomorgado paulomorgado commented Nov 5, 2023

This PR addresses #633 by adding a command-line argument to OpenApiToCsharpCommand /SuppressClientClassesOutput to suppress the output of the generated classes and a command-line argument /SuppressClientInterfacesOutput to suppress the output of the generated interfaces.

The objective here is for the usage of the tooling to be retro-compatible and to be just an additional feature.

If a user wants to generate a client class that implements an interface but not have the interface output as part of the generation, then /GenerateClientClasses:true /GenerateClientInterfaces:true /SuppressClientInterfacesOutput:true.

The same for interfaces. If a user wants to generate a client interface but not have the client class output as part of the generation, then /GenerateClientClasses:true /SuppressClientClassesOutput:true /GenerateClientInterfaces:true /SuppressClientClassesOutput:true.

@paulomorgado paulomorgado force-pushed the omit-client-class-or-interface-output branch from 08abee6 to cfb99a9 Compare November 16, 2023 08:08
@RicoSuter RicoSuter merged commit 5e3e2c0 into RicoSuter:master Nov 21, 2023
1 check passed
@paulomorgado paulomorgado deleted the omit-client-class-or-interface-output branch November 21, 2023 17:39
lahma pushed a commit to lahma/NSwag that referenced this pull request Jan 20, 2024
…ses and interfaces. (RicoSuter#4571)

* Added feature to suppress the generation of the output of client classes and interfaces.

* Added UI for SuppressClientClassesOutput and SuppressClientInterfacesOutput.

* Added documentation.
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.

3 participants