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

Add support for Swagger #923

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from
Draft

Add support for Swagger #923

wants to merge 15 commits into from

Conversation

an-tao
Copy link
Member

@an-tao an-tao commented Jul 8, 2021

No description provided.

@monolidth
Copy link

monolidth commented Sep 6, 2021

I'm very curious about this pr. Thanks for your work!
I appreciate it!


using namespace drogon::internal;

std::pair<std::string, std::string> StructNode::findContentOfClassOrNameSpace(

Choose a reason for hiding this comment

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

Would be good to have checks for comments here, it will break if the class or namespace includes commented uneven brackets

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, you are right, thanks.

Choose a reason for hiding this comment

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

If you want I can help you with that so you can focus on the main dev

Copy link
Member Author

Choose a reason for hiding this comment

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

Great, thank you very much, I look forward to your PR :)

@dieison-depra
Copy link

Any news about PR merge ?

@1111mp
Copy link

1111mp commented Jun 12, 2022

Are there any new developments?

@rbugajewski
Copy link
Collaborator

Are there any new developments?

I think you will know it here first, when there will be additional work done for Swagger support.

PRs are of course always welcome 👍

@nevermoreluo
Copy link

Is there any progress ?

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.

7 participants