Skip to content
This repository has been archived by the owner on Jan 20, 2025. It is now read-only.

doc: add basic java docs and java doc maven plugin #16

Merged

Conversation

nitin-vavdiya
Copy link
Contributor

@nitin-vavdiya nitin-vavdiya commented Oct 12, 2023

Description

Basic Java doc added with Java doc maven plugin

It will create a javadoc.jar file with mvn install command

To 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

@borisrizov-zf
Copy link
Contributor

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.

@borisrizov-zf
Copy link
Contributor

Closes #19

@nitin-vavdiya
Copy link
Contributor Author

suggestions are fixed in new commit

Copy link

@florianrusch-zf florianrusch-zf left a 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.

@nitin-vavdiya
Copy link
Contributor Author

I have not yet completed the review.

Okay, let me know. Will fix all suggestions in one go

@borisrizov-zf
Copy link
Contributor

@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.

@carslen
Copy link
Contributor

carslen commented Oct 17, 2023

@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.

@borisrizov-zf
Copy link
Contributor

@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 :)

@borisrizov-zf
Copy link
Contributor

@nitin-vavdiya please rebase as we've merged two branches.

@nitin-vavdiya
Copy link
Contributor Author

@nitin-vavdiya please rebase as we've merged two branches.

done

@florianrusch-zf
Copy link

@nitin-vavdiya please change the target branch to the develop branch

@nitin-vavdiya
Copy link
Contributor Author

@nitin-vavdiya please change the target branch to the develop branch

there is no develop branch

@borisrizov-zf
Copy link
Contributor

@nitin-vavdiya please change the target branch to the develop branch

yes this repo directly merges into main.

@florianrusch-zf
Copy link

@nitin-vavdiya please change the target branch to the develop branch

there is no develop branch

ahhh sorry, mixed the repository. :-/

@florianrusch-zf
Copy link

But why is then this PR adding the following files? They should be already on the main branch.

image

@borisrizov-zf
Copy link
Contributor

@nitin-vavdiya seems your rebase didn't work correctly. can you double check?

Copy link
Contributor

@borisrizov-zf borisrizov-zf left a 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.

@nitin-vavdiya
Copy link
Contributor Author

Please recheck old comments from Florian, as the rebase has removed some changes.

done

@carslen
Copy link
Contributor

carslen commented Oct 26, 2023

@florianrusch-zf please review if your requested changes have been applied.

@borisrizov-zf
Copy link
Contributor

@florianrusch-zf ping

Copy link

@florianrusch-zf florianrusch-zf left a 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? :-/

@borisrizov-zf
Copy link
Contributor

This can be merged now, with the understanding that we'll improve as we go.

@florianrusch-zf
Copy link

@nitin-vavdiya please apply the requested changes from @borisrizov-zf

If they got applied, we can continue with the merge.

@nitin-vavdiya
Copy link
Contributor Author

@nitin-vavdiya please apply the requested changes from @borisrizov-zf

If they got applied, we can continue with the merge.

Yes, changes are applied

@florianrusch-zf florianrusch-zf changed the title doc: basic java doc and java doc maven plugin added doc: add basic java docs and java doc maven plugin Oct 30, 2023
@florianrusch-zf
Copy link

build pipeline is running into a timeout while trying to pull a dependency. Will retry tomorrow

@borisrizov-zf
Copy link
Contributor

I think the Fraunhofer server is down.

@florianrusch-zf florianrusch-zf merged commit 23fe5e4 into eclipse-tractusx:main Nov 9, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add javadoc to all classes
4 participants