[GitHub] [geode] pivotal-jbarrett commented on issue #4776: GEODE-7642: Removes warnings about missing annotations.

["pivotal-jbarrett (GitHub)" <gi...@apache.org>]

> I was really hoping that the inclusion of `swagger` as compileOnly was a debug mistake, and not something required to remove the warning on our side. Local testing shows that yes, we need Swagger for `compileOnly` to remove the warning :(

Yeah it turns out Swaggers annotations are runtime annotations so anything that loads our Swagger annotated classes need these annotations on the class path. If you look at the geode-core its implementation but optional. This is because we don't have this modularized and unless you are starting the REST interface you don't need Swagger. Once modularized this should clean up all the issues with downstream usage.

[ Full content available at: https://github.com/apache/geode/pull/4776 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org