Skip to content

Conversation

@erunion
Copy link
Member

@erunion erunion commented Nov 9, 2022

🧰 Changes

With v8 of oas-normalize we now have the ability to convert Postman collections to OpenAPI definitions thanks to postman-to-openapi. This work here integrates that work into our API definition upload and validation flow, which means that folks will now be able to upload Postman collections directly into their ReadMe projects.

😎 🛹

@erunion erunion added enhancement New feature or request command:openapi Issues pertaining to the `openapi`, `validate`, `reduce`, or `swagger` commands labels Nov 9, 2022
debug(`attempting to oas-normalize ${file}`);
const oas = new OASNormalize(file, { enablePaths: true });
return oas
.version()
Copy link
Member Author

@erunion erunion Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ended up bringing back .version() in oas-normalize@8.1.0 (readmeio/oas-normalize#227) but slightly changed the original API it had to give us some more information on the file it's been supplied.

@erunion erunion marked this pull request as ready for review November 9, 2022 02:00
@erunion erunion requested a review from kanadgupta November 9, 2022 02:00
@erunion erunion changed the title feat: adding support for Postman collections feat: adding support for uploading and validating Postman collections Nov 9, 2022
Copy link
Contributor

@kanadgupta kanadgupta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small refactor suggestion below but otherwise LGTM!

erunion and others added 3 commits November 9, 2022 10:34
Co-authored-by: Kanad Gupta <8854718+kanadgupta@users.noreply.github.com>
@erunion erunion merged commit 93f80c6 into main Nov 9, 2022
@erunion erunion deleted the feat/postman-support branch November 9, 2022 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:openapi Issues pertaining to the `openapi`, `validate`, `reduce`, or `swagger` commands enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants