search

openapi-tools / swagger-maven-plugin

Maven plugin to activate the Swagger Core library to generate OpenAPI documentation.
MIT License
72 stars 46 forks source link

Add support for skiping generation #20

Closed sunnymoon closed 5 years ago

langecode commented 5 years ago

Thanks for contributing. I will take a look at it asap.

sunnymoon commented 5 years ago

Thanks... If you need any further help please say so.

On Thu, Feb 21, 2019 at 7:07 PM Thor Anker Kvisgård Lange < notifications@github.com> wrote:

Thanks for contributing. I will take a look at it asap.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/openapi-tools/swagger-maven-plugin/pull/20#issuecomment-466125848, or mute the thread https://github.com/notifications/unsubscribe-auth/AEJ6KG3m3COt8RNvKGlKACuEplaXh7rYks5vPu5cgaJpZM4bGjPO .

--

--

Jose Pedro Pereira CTO » Email/Gtalk: jpereira@linkare.com M: +351 210 181 241 <+351%20210%20181%20241> M: +351 918 140 992 <+351%20918%20140%20992> Portugal » T: +351 213 590 623 <+351%20213%20590%20623> A: Av. 5 de Outubro 77, 1º Esq., 1050-049 Lisboa http://www.linkare.pt/pt/contactos.html W: www.linkare.pt Belgium » T: +32 280 842 60 <+32%20280%20842%2060> A: 4 Rue de Pères Blancs, 1040 Etterbeek, Brussels http://www.linkare.be/en/contact.html W: www.linkare.be [image: banner] http://linkare.com/signature_banner_link

sunnymoon commented 5 years ago

Your code currently does not check for "package." but rather "package" with a startsWith, so if I have two packages like a.b.cde.SomeWS and a.b.cdefgh.SomeWS

you'll match both classes if I specify a.b.cde as package and recurse flag is specified. That's why my code had a startsWith(package+".") in the check...

Hope you fix it...