atteo / classindex

Index classes, do not scan them!
Apache License 2.0
263 stars 43 forks source link

Removing dependency to jaxb-api #49

Closed Yannic92 closed 5 years ago

Yannic92 commented 5 years ago

As far as I see the only reason for the jaxb-api dependency is that it is used in a JavaDoc comment. Wouldn't it be better to exclude this from dependencies? It doesn't seem to be worth adding a dependency just to mention it in a JavaDoc comment.

It might be possible that I just didn't see where you're actually using this dependency.

sentinelt commented 5 years ago

Fixed. I will release new version shortly. Thanks.

Yannic92 commented 5 years ago

Great, thank you! We plan to use your library in eclipse ditto and this saves us a lot of excludes :)