Skip to content

Minor Javadoc fixes #1002

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 12, 2019
Merged

Minor Javadoc fixes #1002

merged 2 commits into from
Jul 12, 2019

Conversation

MakarovS
Copy link
Contributor

@MakarovS MakarovS commented Jul 11, 2019 •
edited
Loading

Overview

Fix some minor Javadoc issues.
In Node I removed @throws IllegalStateException if the node proxy is closed as it's not applicable to NodeFake.

Definition of Done

  • There are no TODOs left in the code
  • Changelog is updated if needed (in case of notable or breaking changes)
  • The continuous integration build passes

@coveralls
Copy link

coveralls commented Jul 11, 2019 •
edited
Loading

Coverage Status

Coverage remained the same at 85.658% when pulling a2c6817 on minor_javadoc_fix into 283e4e4 on master.

@dmitry-timofeev dmitry-timofeev merged commit 705a1ab into master Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bullet-tooth bullet-tooth Awaiting requested review from bullet-tooth

1 more reviewer

@dmitry-timofeev dmitry-timofeev dmitry-timofeev approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MakarovS @coveralls @dmitry-timofeev