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
Prepare Javadoc for Java8 #1653
Conversation
The regex to find them is: (^\s*\*\s@author\s+[\w\s'-]+)\s+<[\w@.]+> replaced with $1
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
I fixed all javadoc errors. Let's see how it goes with those super-strict settings. We might want to consider setting the |
I put out an announcement asking if anybody objects to just removing the author tags entirely: http://forum.terasology.org/threads/removal-of-all-javadoc-author-tags-any-objections.1273/ Pinging @MovingBlocks/contributors on here to get the word out. If anybody objects please post here or in the forum thread (preferred) :-) We have a nice recently updated JavaDoc page in the wiki too: https://github.com/MovingBlocks/Terasology/wiki/JavaDoc I figure we can add the javadoc task to the PR builder job so any failures will break the build and report back to a PR that something is wrong with javadoc. Then we can get it fixed before merging. Really we should probably enable the checkstyle nagging for javadoc and such while we're at it (not necessarily in this PR), and tweak some rules to see if we can adhere to a convention. Doing some build testing with http://jenkins.terasology.org/job/TerasologyTemp/ - looks like it works 100% with Java 8! (the earlier build failed from a timeout) |
Merged this since we'll needs these fixes regardless. Then we can do the author tags next - no objections so far. |
This PR fixes most of the Javadoc errors. A few more tricky ones remain and will be dealt with separately.
I left the
@author
tags in, but removed the email address hoping that this approach suits all or at least most developers.