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

Fixes #1535: Add Javadocs for Post.java #1536

Merged
merged 1 commit into from
Oct 6, 2017

Conversation

vibhcool
Copy link
Member

@vibhcool vibhcool commented Sep 15, 2017

Fixes:-

  1. issue Add Javadocs for Post.java #1535
  2. improves code readability

Short description

I have:

  • There is a corresponding issue for this pull request.
  • Mentioned the Issue number in the pull request commit message Fixes #<number> commit message
  • There is only strictly only one commit per issue.

For the reviewers

I have:

  • Reviewed this pull request by an authorized contributor.
  • The reviewer is assigned to the pull request.

@vibhcool
Copy link
Member Author

PR is open for review

@codecov-io
Copy link

codecov-io commented Sep 15, 2017

Codecov Report

Merging #1536 into development will not change coverage.
The diff coverage is 13.33%.

Impacted file tree graph

@@              Coverage Diff              @@
##             development   #1536   +/-   ##
=============================================
  Coverage           9.27%   9.27%           
  Complexity           424     424           
=============================================
  Files                208     208           
  Lines              19230   19230           
  Branches            3532    3532           
=============================================
  Hits                1783    1783           
  Misses             17129   17129           
  Partials             318     318
Impacted Files Coverage Δ Complexity Δ
src/org/loklak/harvester/Post.java 26.98% <13.33%> (ø) 4 <1> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f9b1565...6486296. Read the comment docs.

@vibhcool vibhcool force-pushed the add_post_docs branch 4 times, most recently from 49b3658 to 80d602e Compare October 2, 2017 17:02
@mariobehling
Copy link
Member

Cheers!

@mariobehling mariobehling merged commit c398d0f into loklak:development Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants