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

Improve javadoc for all addons #15667

Merged
merged 3 commits into from
Sep 30, 2023
Merged

Improve javadoc for all addons #15667

merged 3 commits into from
Sep 30, 2023

Conversation

holgerfriedrich
Copy link
Member

The javadoc was quite broken form most of the addons.

This PR fixes most of the errors the javadoc run is reporting.

Sorry for bothering almost everybody with this review request.

Fix broken references, syntax errors.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the huge cleanup! 👍

Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Did not check all 600+ files, but random looked at 100 or so.

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a minor comment from my side. Thanks for the cleanup, and @wborn, thanks for reviewing.

@lolodomo
Copy link
Contributor

Except sonos binding, changes in astro, freebox, freeboxos, lgwebos, netatmo, ntp, powermax, pulseaudio, sonyprojector, weatherunderground looks fine to me. I have not checked the changes in other bindings.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Copy link
Member

@GiviMAD GiviMAD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for broadlinkthermostat, unifiedremote and googlestt bundles. Thank you!

@wborn wborn merged commit cbf4411 into openhab:main Sep 30, 2023
2 checks passed
@wborn wborn added this to the 4.1 milestone Sep 30, 2023
@holgerfriedrich holgerfriedrich deleted the pr-sat branch September 30, 2023 20:31
pat-git023 pushed a commit to pat-git023/openhab-addons that referenced this pull request Oct 13, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 21, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 29, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: querdenker2k <querdenker2k@gmx.de>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
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.