-
Notifications
You must be signed in to change notification settings - Fork 154
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
Critical: Build blocked by custom taglets #909
Comments
Looking back I think I have most of the patch for JDK11 taglets, but had issues with some of them. I'll look tomorrow with a fresh head but I still think simpler would be better here. |
Build is ok, I would still like your view @egonw on inlining the data in the doc. At the moment I find it less convenient to add a citation for example. We have a bespoke JavaDoc process which needs special treatment. Which is a bit of a pain, keeping things simple is more maintainable long term. On a related note the new JDK 17 javadoc layout is very package focused and highlights the need for some restructuring - ala #626 |
I do think keeping things simple usually pays off in the long run. Whenever I used niche tools/functionality most caused painful issues at one point or where just not supported any longer (which then makes you drop them or migrate to sth else). Inlining the citations using Plain Old JavaDoc/HTML probably makes them less fun to write, but I'd still consider this a preferred solution if it makes things simpler. |
The introduction of the Taglets at the time actually simplified and improved our documentation. It solve a number of issue:
There are two common pattern in our use of taglets:
For example:
So, generally, they made things easier, sometimes communication to users, sometimes the development of the CDK. |
I think the module/githash one as actually still potentially useful as otherwise you can't know in the JavaDoc where stuff is. Aligning the packages with the modules would make this less of an issue. I'm kind of happy I got things working but I still maintain it's an extra headache/inconvenience for building the project. I should try and get cdk-build-util on maven central though. |
I can help with that. |
I think should be easy since we already have the org.openscience.cdk domain working. Just needs a bit of cleanup, remove the EBI repo etc. It does still work but central is obv better. |
I do remember having a bit of an issue downloading from the EBI repo when building the CDK some months ago. |
Just tried to do a release but got stuck with the custom taglets... (cdk-build-util) which I've been putting off updating. The APIs for all of these changed in Java 9+, so basically it all needs a rewrite. IIRC some of the functionality is now gone completely (or at least very poorly documented) and so it is a non trivial task to update. I certainly don't have time. Now previously I would just do a release with Java 8 and all was OK - however we now have a Java 11 module (iordf) due to JENA needing it.
My preferred solution would be to drop all the custom JavaDoc stuff, it's been a pain for a long time. However there are some nice to have features, like the cite, dictref stuff. The bugs tag is kind of redundant IMHO but sure I guess also "nice to have". What I think makes more sense is inline the data from these tags as Plain Old JavaDoc/HTML (perhaps leaving some tag in place if someone ever wants to put reverse it and them back in).
For example:
would become
The text was updated successfully, but these errors were encountered: