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

Swagger 2.0 Support #219

Open
wedtm opened this issue Feb 24, 2015 · 38 comments
Open

Swagger 2.0 Support #219

wedtm opened this issue Feb 24, 2015 · 38 comments

Comments

@wedtm
Copy link

wedtm commented Feb 24, 2015

Is this possible yet?

@calfzhou
Copy link
Contributor

calfzhou commented Mar 4, 2015

really need 2.0 support, swagger 2.0's authorization supports oauth2 4 flows, but only 2 flows for version 1.2.

@dblock
Copy link
Member

dblock commented Mar 4, 2015

I'd really like to see https://github.com/tim-vandecasteele/grape-swagger/issues/137, in which we rewrite grape-swagger using a proper swagger DSL library instead of keeping at hacking on stuff. That brings us to Swagger 2.0. And anyone who does it will be my personal hero.

@tbeauvais
Copy link

👍

1 similar comment
@danielkummer
Copy link

👍

@florrain
Copy link

florrain commented Jul 1, 2015

Any news about that?

👍
👍
👍

@yairgo
Copy link

yairgo commented Jul 9, 2015

Has anyone thought about this at all yet? Is there a path going forward that we can get started and maybe have multiple people work on it at once?

@dblock
Copy link
Member

dblock commented Jul 10, 2015

Doesn't look like anyone has done anything about this, the first one wins!

@birbird
Copy link

birbird commented Jul 25, 2015

How about this
https://github.com/Gild/ruby-swagger

@dmitry
Copy link
Contributor

dmitry commented Jul 25, 2015

As I understand it's not generating documentation on the fly. But it has support for grape: https://github.com/Gild/ruby-swagger/tree/master/lib/ruby-swagger/grape

@openmosix
Copy link

@dmitry correct, there is a rake task - extracting the data from grape into a YML format - this allows tech editors to write all the doc outside the annotations. The documentation can be compiled into its final swagger.json form - this allow us to host the documentation on CDN rather than from the API server.

@aceunreal
Copy link

👍

2 similar comments
@korstiaan
Copy link
Contributor

👍

@wagner
Copy link

wagner commented Oct 8, 2015

👍

@Micka33
Copy link

Micka33 commented Nov 17, 2015

would be awesome.

@johnhamelink
Copy link

👍

@rngtng
Copy link
Contributor

rngtng commented Dec 2, 2015

/paging @oliverzeyen

@herberzt
Copy link

Is the only current strategy to get 2.0 Swagger compliance still https://github.com/Gild/ruby-swagger? Are there any plans to add 2.0 support to this gem?

@dblock
Copy link
Member

dblock commented Dec 29, 2015

Nobody is actively working on this @herberzt. Maybe you will?

@edbo
Copy link

edbo commented Jan 11, 2016

Can one of the original maintainers outline how someone would go about this task?

Right now, for someone new using this project, it's not necessarily obvious how to go about starting to do this.

@dblock I'm not suggesting you take this task up but it's currently a bit daunting for someone that hasn't maintained this repo before to take up and it's definitely something I'd like to see happen.

@dblock
Copy link
Member

dblock commented Jan 11, 2016

I am here to help. What I need is a passing build in a pull request, and I can definitely carry it forward after it - #311 was very promising.

@dblock
Copy link
Member

dblock commented Jan 22, 2016

#311 has been merged onto a branch called swagger-2.0. Looking forward to more PRs to move that forward!

@pswenson
Copy link

where do things stand wtih the 2.0 fork and getting it merged in: https://github.com/LeFnord/grape-swagger ?

We are trying it out, but it's unclear where things stand.

@XBeg9
Copy link

XBeg9 commented Feb 29, 2016

👍

@dblock
Copy link
Member

dblock commented Mar 9, 2016

So we have a 2.0 branch. Who wants to take ownership, get that merged onto master, released, etc.? What remains to get done? @LeFnord maybe?

@LeFnord
Copy link
Member

LeFnord commented Mar 9, 2016

Hi guys … I would prefer to create a new grape-swagger-2 repo under the ruby-grape project, so much more people can contribute to it and it feels more on the right place.
An side effect of this would be, that we can start a discussion on decisions, cause I don't want to decide all alone, think this would not be a good idea 😉 … there is something to decide, e.g. how to handle grape specific behavior → exactly_one_of, mutually_exclusive to name only some …

@dblock
Copy link
Member

dblock commented Mar 10, 2016

I am not in love with the idea of two gems for different versions of the spec. I think people just want to upgrade to the swagger 2.0 spec soon and call it a day, do you see a reason one wouldn't want to?

@LeFnord
Copy link
Member

LeFnord commented Mar 10, 2016

@dblock I hope and I wish … then it would be best as it is for the moment, or?
or did you prefer to use my fork for that, this is also ok for me …

@dblock
Copy link
Member

dblock commented Mar 10, 2016

I'd like to do the following:

  1. Release 0.10.5.
  2. Branch 0.10.5 into 0.10.x, continue releasing bug fixes only in a 0.10.x series compatible with 1.5
  3. Merge swagger-2.0 onto master, release 0.11.0 with only Swagger 2.0 support.
  4. Move forward.

Thoughts?

If you agree with this @LeFnord I want you to take care of (1), (2), (3) and (4). Aka all of it. I've not been actively developing this library I have been just cheerleading the effort here and would love for more people to take a very active role. Who can be better than the person who wrote the majority of 2.0 support for doing that? :)

@LeFnord
Copy link
Member

LeFnord commented Mar 11, 2016

Hi @dblock mmh ok, but for (1)..(3) I need your advise, cause I thing I haven't the rights to do so, and yeap it also would be the first time 😉

point (3) would prefer to start with tag 0.20.x, so the other branch has enough version space for changes,
and it indicates a little more breaking changes (ok, doing it right SemVer, we have to use 1.0.0)
point (4) definitely

@dblock
Copy link
Member

dblock commented Mar 12, 2016

@LeFnord I invited you the swagger-maintainers org. I am happy to help with whatever you need.

You should PR yourself under grape-swagger in https://github.com/ruby-grape/ruby-grape.github.io/blob/da76718ff723d8e6e6a3f45972045ce133220842/team/index.md.

Drop me an email at dblock[at]dblock[dot]org with your Rubygems e-mail address for access to that, since you're making the next release.

We should introduce proper semantic versioning, whatever that means, trust your judgement on which version this should be. Also cc: @tim-vandecasteele and @CraigCottingham wrt ^^^ who are the other two maintainers on this project.

Finally, thank you for spearheading this.

@florrain
Copy link

Looks like you're working out a solution, that's awesome!

Do you have a rough idea in terms of release timeline (either in days/weeks/months)? Like a first beta release that the public could test?

@LeFnord
Copy link
Member

LeFnord commented Mar 14, 2016

@florrain the release is in discussion …
but for first one could use the swagger-2.0 branch and give feedback,
or much more preferred commit fixes 😉

@florrain
Copy link

Got it, I'm seeing exceptions using that branch. Should I create an issue on this repo or do you have a discussion channel I can hop in?

@LeFnord
Copy link
Member

LeFnord commented Mar 14, 2016

I personally prefer issues … or some slack lobby

@dblock
Copy link
Member

dblock commented Mar 14, 2016

(Maybe we should move the mailing list to slack, I wonder ...) I opened ruby-grape/ruby-grape.github.io#29 if anyone feels strongly comment there.

@florrain
Copy link

Created an issue for you @LeFnord. Down for Slack but maybe as a complementary tool for discussions between maintainers and direct/quick support, not a replacement of the mailing list even though I'm personally not a fan of the concept.

@LeFnord
Copy link
Member

LeFnord commented Mar 14, 2016

yeap, definitely not a replacement of a mailing list

@dblock
Copy link
Member

dblock commented Mar 15, 2016

FYI I created issue labels for swagger 1.2 and 2.0, should help out with sorting.

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

No branches or pull requests