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

Add support for draft-6 examples #22

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
sixlive wants to merge 3 commits into wework:master
base: master
Choose a base branch
Loading
from sixlive:add-draft-6-examples

Conversation

Copy link

@sixlive sixlive commented Feb 15, 2019

Description

This adds support for draft-6 examples. Since OpenAPI only supports single examples this transform takes the first example from the array of examples and sets that as the value. I've been using my implementation with speccy for a few days and its been working great and is a must have for my workflow.

Resolves #20

Solosneros, brandonandre, and easybird reacted with thumbs up emoji
Copy link

@sixlive @philsturgeon Hey, looking forward to that PR. Is there anything I can do to help? :)

Copy link
Author

sixlive commented Mar 4, 2019

Is there any way we can get an update on this?

Copy link

@philsturgeon Can we merge and release this? I could use it in Prism.

Copy link
Contributor

@XVincentX I don't work at WeWork anymore. I work at stoplight.io, with you!

@djtarazona hey there! Do you know who is looking after this? If it's nobody can you ask some higher up to make me a collaborator or give me the package? It's a shame to see this getting ignored.

Copy link

I thought you were still a collaborator with privileges!

Copy link
Contributor

@philsturgeon philsturgeon left a comment

Choose a reason for hiding this comment

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

I can approve but I think I'm a random person.

Copy link

@djtarazona hey there! Do you know who is looking after this? If it's nobody can you ask some higher up to make me a collaborator or give me the package? It's a shame to see this getting ignored.

I don't know who's looking after this. @iainmcgin can you help out here?

Copy link
Contributor

I would be very happy to take this over btw, and keep it maintained for as long as it’s required. I’m working on closing the OpenAPI and JSON Schema divergence gap through a few different means with the two projects themselves but until then this tool is really important, and shouldn’t be left to stagnate. I know Pascal said he’d have to check with the lawyers before giving code away, so if we could get the ball moving on that that’d be great! :)

Copy link

could this be merge or is no longer maintained?

@sixlive sixlive dismissed stale reviews from XVincentX and philsturgeon via 441c117 July 11, 2019 16:07
Copy link
Contributor

Not being maintained at all, see https://github.com/openapi-contrib/json-schema-to-openapi-schema instead.

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

@philsturgeon philsturgeon philsturgeon left review comments

@XVincentX XVincentX XVincentX left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrading to support examples from draft-06

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