Skip to content
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

Java 8 compliant Javadoc #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

snodnipper
Copy link
Contributor

The Java 8 doc lint is pretty strict with Javadoc.

This commit should resolve problems that result in failure and help us avoid having to turn off the lint tool.

Please comment on any TODOs and I can rebase.

@snodnipper snodnipper changed the title PJava 8 compliant Javadoc Java 8 compliant Javadoc Oct 11, 2015
@ColeFerrier
Copy link

Would it be possible to get this merged before the next release? this might need updated due to changes on master? what is Esri's thoughts on getting this merged?

@randallwhitman
Copy link
Contributor

Unfortunately, conflicts have arisen - that would have to be resolved.

@alocke @stolstov Had you any comments on the changes?

@snodnipper
Copy link
Contributor Author

perhaps if folk could raise any comments with the view to merge then I could rebase.

@stolstov
Copy link
Member

This fixes will get in eventually, we'll try to make it for next release (after 1.3). They will have to be merged into another branch.

@stolstov
Copy link
Member

@snodnipper It's just we have certain internal code that needs to be updated manually. I'll try to make the process easier.

@monezz
Copy link

monezz commented Jun 27, 2016

you could add this property to the pom in the main branch as quick fix:
<additionalparam>-Xdoclint:none</additionalparam>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants