-
Notifications
You must be signed in to change notification settings - Fork 16
doc: add basic java docs and java doc maven plugin #16
doc: add basic java docs and java doc maven plugin #16
Conversation
Hi @nitin-vavdiya, can you update the contents of the docs to some of the key components? I think we can fix the rest as we go. |
Closes #19 |
src/main/java/org/eclipse/tractusx/ssi/examples/BuildDIDDocEd25519VerificationKey2020.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/examples/BuildDIDDocJsonWebKey2020.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/examples/ResolveDIDDoc.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/proof/hash/LinkedDataHasher.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/proof/transform/LinkedDataTransformer.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/proof/types/jws/JWSProofVerifier.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/serialization/SerializeUtil.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/serialization/jsonLd/JsonLdValidator.java
Outdated
Show resolved
Hide resolved
suggestions are fixed in new commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not yet completed the review.
src/main/java/org/eclipse/tractusx/ssi/examples/ResolveDIDDoc.java
Outdated
Show resolved
Hide resolved
Okay, let me know. Will fix all suggestions in one go |
@carslen do we need an IP check? The handbook defines "project content" as any kind of content. Although this is just class documentation, it probably still falls under the >1000 SLOC rule. |
I did not check the hole PR, but if it is only comments, then I'd approve the PR without starting a IP-Check. |
Yes, it's only Javadoc on each class. Thanks for the help :) |
@nitin-vavdiya please rebase as we've merged two branches. |
done |
@nitin-vavdiya please change the target branch to the develop branch |
there is no develop branch |
yes this repo directly merges into main. |
ahhh sorry, mixed the repository. :-/ |
@nitin-vavdiya seems your rebase didn't work correctly. can you double check? |
6d33461
to
26951b5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please recheck old comments from Florian, as the rebase has removed some changes.
src/main/java/org/eclipse/tractusx/ssi/examples/ResolveDIDDoc.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/IKeyGenerator.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/jwk/JsonWebKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/model/base/Base64.java
Outdated
Show resolved
Hide resolved
done |
@florianrusch-zf please review if your requested changes have been applied. |
@florianrusch-zf ping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's way to much to do a proper review on this...I would say, lets merge it and improve the comments, when revisiting the places in later changes. WDYT? :-/
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/IPrivateKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/IPublicKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/jwk/JsonWebKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/jwk/JsonWebKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/crypt/x21559/x21559PublicKey.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/did/resolver/DidDocumentResolverRegistry.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/did/resolver/DidDocumentResolverRegistry.java
Outdated
Show resolved
Hide resolved
This can be merged now, with the understanding that we'll improve as we go. |
@nitin-vavdiya please apply the requested changes from @borisrizov-zf If they got applied, we can continue with the merge. |
Yes, changes are applied |
build pipeline is running into a timeout while trying to pull a dependency. Will retry tomorrow |
I think the Fraunhofer server is down. |
Description
Basic Java doc added with Java doc maven plugin
It will create a javadoc.jar file with
mvn install
commandTo verify Java doc run
./mvnw javadoc:javadoc
Java dock verification was added with compile goal, which means if there is any issue in Java doc then the code will not compile.
Closes #19