-
Notifications
You must be signed in to change notification settings - Fork 63
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
Fix javadoc errors and warnings for Kitodo-API #6162
base: master
Are you sure you want to change the base?
Conversation
After these fixes, the next errors and warnings are reported for Kitodo-DataManagement. |
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
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.
HH:MM:SS.mmm is a better description for the expected time format than showing the regular expression.
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Can this PR be merged? Then I could work on the Javadoc warnings and errors for other modules of Kitodo. |
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Kitodo-API/src/main/java/org/kitodo/api/dataformat/MediaPartial.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Stefan Weil <[email protected]>
Signed-off-by: Stefan Weil <[email protected]>
Suggested-by: Henning Gerhardt <[email protected]> Signed-off-by: Stefan Weil <[email protected]>
Signed-off-by: Stefan Weil <[email protected]>
Many of the other javadoc errors are similar to this one:
Do you have a suggestion how this should be fixed? Would using "→" be okay? |
How can we proceed with this topic? |
@solth, is there still anything that prevents this PR from being merged? |
This partially fixes issue #6161.