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

Include Headers in Swagger generator #624

Closed
wants to merge 1 commit into from

Conversation

jlebrijo
Copy link

This PR includes Apipie headers in Swagger json.

@iNecas
Copy link
Member

iNecas commented Jun 26, 2018

Tests are failing currently. Also, could you elaborate more on what's the added value for this, as you were explaining it to somebody that doesn't do Swagger on day2day basis (such as me:)

@Uysim
Copy link
Contributor

Uysim commented Sep 7, 2018

I wish this can merge every soon

@iNecas
Copy link
Member

iNecas commented Sep 7, 2018

It can't until the tests are passing: if you want, you can have a look at what's missing and send additional PR to fills the gaps. That's the safest way to make your wish come true in open-source :)

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.

3 participants