-
Notifications
You must be signed in to change notification settings - Fork 21
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
fix: remove unused cli flags #730
fix: remove unused cli flags #730
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe that the includeInternal
flag was in the original v1 of openapi-generator
, and then was never implemented for v2
.
Removes unused command line flags to enshrine existing behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't suggest an edit in that location, but the flag
and boolean
imports are now unused
The `--internal` and `--unstable` cli flags were never actually used by openapi-generator. This commit removes no functionality of the tool, it only cleans the cli interface. Ticket: VL-1855
2fbadf3
to
5a8a659
Compare
🎉 This PR is included in version @api-ts/[email protected] 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
🎉 This PR is included in version @api-ts/[email protected] 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
🎉 This PR is included in version @api-ts/[email protected] 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
The
--internal
and--unstable
cli flags were never actually used by openapi-generator. This commit removes no functionality of the tool, it only cleans the cli interface.Ticket: VL-1855