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? |
6f9094d
to
45d6c50
Compare
…ator Signed-off-by: Muhammad Aaqil <aaqilcs102@gmail.com>
45d6c50
to
04d39b7
Compare
Signed-off-by: Muhammad Aaqil <aaqilcs102@gmail.com>
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 ? |
Signed-off-by: Muhammad Aaqil <aaqilcs102@gmail.com>
This PR is related to |
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 👈