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

[zoneminder] Use DS annotations, fix static code analysis findings #3382

Merged
merged 1 commit into from
Jan 26, 2019

Conversation

wborn
Copy link
Member

@wborn wborn commented Mar 20, 2018

  • Use DS annotations
  • Fix static code analysis findings

Copy link
Member

@martinvw martinvw left a comment

Choose a reason for hiding this comment

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

@Mr-Eskildsen are you okay with these changes?

@Mr-Eskildsen
Copy link

They seems to be harmless :-)
I am currently working on a major rework of the Binding. So I guess it would be important for me to try to embed the changes in the reworked code. I think the merge will handle some of it, but I would need to manually go through the code. It might be helpfull for this process if you can tell me how to perform this static code analysis.

@wborn
Copy link
Member Author

wborn commented Mar 20, 2018

I am currently working on a major rework of the Binding.

Thats great news @Mr-Eskildsen! You might want to create a WIP PR for that to increase visibility. So far I have not added DS annotations and fixed static code analysis for bindings that already have pending major rework PRs.

It might be helpfull for this process if you can tell me how to perform this static code analysis.

When you build the project with Maven (mvn clean install) the result of the static code analysis is available as HTML in: target/code-analysis/report.html

@Mr-Eskildsen
Copy link

When you build the project with Maven (mvn clean install) the result of the static code analysis is available as HTML in: target/code-analysis/report.html

OK, I will look at that as well. My main problem right now, is time, but I hope to get the last things done in the next month or so. It might be easier to postpone the above change? Else it seems quite harmless, so it isn't important :-)

@wborn
Copy link
Member Author

wborn commented Sep 22, 2018

FYI, this PR is waiting for #3583 by @Mr-Eskildsen.

@wborn wborn changed the title [ZoneMinder] Use DS annotations, fix static code analysis findings [zoneminder] Use DS annotations, fix static code analysis findings Dec 18, 2018
@wborn wborn force-pushed the zoneminder-improvements branch from 9b2585d to e627df3 Compare January 25, 2019 20:41
@wborn
Copy link
Member Author

wborn commented Jan 26, 2019

@wborn wborn merged commit 86c6f1f into openhab:master Jan 26, 2019
@wborn wborn deleted the zoneminder-improvements branch January 26, 2019 10:56
@wborn wborn removed the awaiting other PR Depends on another PR label Jan 26, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Jun 19, 2019
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
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.

3 participants