-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: enable openapi generator to include & exclude specific controllers #10204
base: master
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 8358248924Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
2cd0620
to
55fb7ed
Compare
Hi, @dhmlau Can you please have a look at this PR and suggest to me any changes/improvements required? |
45d6c50
to
04d39b7
Compare
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.
Do you mean the generator would generate the routes but the APIs would not be accessible? |
I think I am misunderstanding this PR. Which cli command is this referring to ? |
This PR is related to |
17dde51
to
9757274
Compare
…ator Signed-off-by: Muhammad Aaqil <[email protected]>
9757274
to
b731705
Compare
At this point, the openapi generator has no way to include or exclude any API. This PR enables the openapi generator to receive three new options for users to control what APIs should be generated.
readonly
(boolean): Passing this option would make the openapi generator create only GET endpoints.exclude
(regex - string): This enables the user to exclude particular APIs from the end result.include
(regex - string): This enables the user to include only some of the APIs and exclude everything else from the end result.Checklist
npm test
passes on your machinepackages/cli
were updatedexamples/*
were updated👉 Check out how to submit a PR 👈