search

javanile / yamlinc

Compose multiple YAML files into one with $include tag. Split Swagger/OpenAPI into multiple YAML files.
MIT License
114 stars 22 forks source link

Support for STDOUT and specified file output #7

Closed justinnichols closed 6 years ago

justinnichols commented 6 years ago

This is the implementation of the enhancement in #2 This does not currently verify that the target output file name isn't within the set of input files (either the named input file or the derived input files via $include).

justinnichols commented 6 years ago

Tests were fixed with this PR. I did not add a specific test for --output (or -o) as I didn't have time to continue working on this. I hope this is good enough for now, and thanks!

justinnichols commented 6 years ago

There may be an issue specifying an output target file that is within a folder path and not directly in the working directory. I'll work on this as I'm able and will update this PR.

justinnichols commented 6 years ago

Okay, I think this is ready. Please critique as you'd like :) Thanks again!

francescobianco commented 6 years ago

I love your work! Thanks for your support and your time spent. :+1:

justinnichols commented 6 years ago

Thanks for merging! Glad I could help :)

jahluwalia commented 5 years ago

I see this was merged, but when will it be updated on npm?

francescobianco commented 5 years ago

Hi @jahluwalia @justinnichols a new version 0.1.0 is on NPM registry https://www.npmjs.com/package/yamlinc with: Support for STDOUT

please help, support Yamlinc: