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

Add some docs for core classes #1641

Closed

Conversation

Bluefire2
Copy link

Add javadocs for {Metadata, Row} x {Reader, Writer, Deleter}. We also removed public from interface methods since it's redundant (interface methods can't have any other access level in Java 8).

Signed-off-by: Kirill Chernyshov <kc875@cornell.edu>
@coveralls
Copy link

coveralls commented Dec 13, 2019

Coverage Status

Coverage decreased (-7.7%) to 38.629% when pulling 20da794 on aniroodh-ravikumar:some-database-docs into cf00ff8 on locationtech:master.

@Bluefire2
Copy link
Author

How could the coverage decrease (or change at all), if only docstrings are changed? Was there a concurrent PR that had been merged?

@jdgarrett
Copy link
Contributor

Looks like the formatter needs to be run on the updated classes before the build will pass.

@jdgarrett
Copy link
Contributor

Changed files can be formatted by running mvn clean install from the GeoWave directory. Once the files in question have been run through the formatter, the build will pass and this PR can be merged.

@jdgarrett
Copy link
Contributor

I have rolled the changes from this PR into #1669.

@jdgarrett jdgarrett closed this Feb 12, 2020
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.

4 participants