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

☄ New command line arg: additional-data #1772

Merged
merged 8 commits into from
Aug 8, 2022

Conversation

PureKrome
Copy link
Contributor

Resolves #1573

  • new command line arg: --additional-data [true|false] (or -ad [true|false]). Default is true (which is the current status quo).

Here's an example which does NOT add the additional data to any generated models:

--openapi https://petstore.swagger.io/v2/swagger.json --additional-data false --language csharp -o PetshopClient -c ApiClient --log-level information --clean-output true

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the contribution!

A few minor changes. Also, can you add an entry in the changelog (under unreleased, added) please?

docs/using.md Outdated Show resolved Hide resolved
docs/using.md Outdated Show resolved Hide resolved
src/kiota/KiotaHost.cs Outdated Show resolved Hide resolved
@PureKrome PureKrome requested a review from baywet August 8, 2022 14:43
@PureKrome
Copy link
Contributor Author

Done 💪🏻 Ready for another review 👀 plz

@baywet baywet merged commit 054e8bc into microsoft:main Aug 8, 2022
@baywet baywet self-assigned this Aug 8, 2022
@baywet baywet added enhancement New feature or request generator Issues or improvements relater to generation capabilities. labels Aug 8, 2022
@PureKrome
Copy link
Contributor Author

Thank you @baywet ❤️ 🤗

@PureKrome PureKrome deleted the include-additional-data branch August 8, 2022 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request generator Issues or improvements relater to generation capabilities.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to -not- generate AdditionalData property
2 participants