Skip to content
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

2022-01-01 One CLI several Open API files several supported output formats #29

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

VRDate
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.

@davidmoten
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).

@VRDate
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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants