ideaconsult / jna-inchi

Wrapper to access InChI from Java
GNU Lesser General Public License v2.1
0 stars 0 forks source link

add javadocs #7

Closed uli-f closed 1 year ago

uli-f commented 1 year ago

Overall, the javadoc documentation is helpful and sufficient.

I went over all classes and had a look at the javadocs. I added/amended bits and pieces myself.

There are a few javadocs that might benefit from your knowledge of the codebase and the rinchi library:

uli-f commented 1 year ago

Probably best to merge PR #3 before working on this, as I have already done quite a bit.

ntk73 commented 1 year ago

I will see for these javadocs improvements. And yes, after merging your latest pull request.

ntk73 commented 1 year ago

Javadocs are ready

uli-f commented 1 year ago

Looks good to me 👍🏼