Skip to content

Fix to not use "no-" prefix in options #1121

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

Merged
merged 5 commits into from
Mar 25, 2025
Merged

Conversation

takayukioda
Copy link
Contributor

Fixes #1117

It seems an option with "no-" prefix will be treated as special case. unjs/citty#176 (comment)

Note that some how option text in my local environment keep showing me '--no-client' even though I removed it from the code. I haven't dig deeper for this.

OPTIONS

      -p, --path (required)    path/url to swagger scheme                                                                                                                              
          -o, --output="./"    output path of typescript api file                                                                                                                      
...                                                                                                           
                --no-client    generate an API class # I don't know why but this shows "no-client" option instead of "client" in my local
...                       
              --sort-routes    sort routes in alphabetical order                                                                                                                       
            --custom-config    custom config: primitiveTypeConstructs, hooks, ...                                                                                                      

Copy link

changeset-bot bot commented Mar 25, 2025

🦋 Changeset detected

Latest commit: e43437f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
swagger-typescript-api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

_
Signed-off-by: Sora Morimoto <sora@morimoto.io>
Signed-off-by: Sora Morimoto <sora@morimoto.io>
@smorimoto smorimoto added the bug Something isn't working label Mar 25, 2025
@smorimoto smorimoto merged commit 5eadf67 into acacode:main Mar 25, 2025
1 check passed
@github-actions github-actions bot mentioned this pull request Mar 25, 2025
@takayukioda takayukioda deleted the patch-1 branch March 25, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generating with --no-client flag includes a client
2 participants