Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

2022年01月01日 One CLI several Open API files several supported output formats #29

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

Open
VRDate wants to merge 11 commits into davidmoten:master
base: master
Choose a base branch
Loading
from CLK-AL:2022年01月01日

Conversation

Copy link

@VRDate VRDate commented Jan 2, 2022

Updated README.md pointing to relevant tests with more examples.
Improved converter and command line to support several Open API files in one directory or one Open API file as well as several supported output formats. Updated tests adding new edge cases and validations.

tomer-sogo and others added 11 commits January 2, 2022 16:05
...irectory ot one file as well as several supported output formats
removed duplicate properties
removed duplicate properties version 2022年01月23日-SNAPSHOT
Copy link
Owner

Thanks for the work though it's usually better to discuss first. It's pretty easy to do what you are suggesting with some simple bash scripting. Why would we want to add to the complexity of the library? (I haven't inspected your PR closely yet but I am curious).

Copy link
Author

VRDate commented Mar 28, 2022

Using bash script will be less efficient since a JVM will be created for each file it should consume more memory and IO then using only java,
I just liked your work and wanted to share mine.

davidmoten reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /