-
-
Notifications
You must be signed in to change notification settings - Fork 685
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
Update guidelines.md #933
Update guidelines.md #933
Conversation
* Remove eclipse smarthome references. * More concise * Add examples for logging * Merge null annotations part into this file * Use MUST instead of SHOULD in many places, where we nowadays enforce such rules * OSGi framework level R5 Signed-off-by: David Graeff <david.graeff@web.de>
Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits). Built with commit 304ef79 https://deploy-preview-933--openhab-docs-preview.netlify.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great update and this information is needed. Here are some suggestions for polishing it up a bit.
review comments Signed-off-by: David Graeff <david.graeff@web.de>
For openhab#933 Signed-off-by: David Gräff <david.graeff@web.de>
For #933 Signed-off-by: David Gräff <david.graeff@web.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far.
Thanks to @openhab-5iver for the content review.
I have added some minor style comments below.
Co-Authored-By: davidgraeff <david.graeff@web.de>
review comments Signed-off-by: David Graeff <david.graeff@web.de>
Comments addressed |
This was one big blob of text. I have added some life to it with examples and an extension directory layout etc.
The most controversial point is OSGi R5, but I think it is about to be time. We are running openHAB on R6 already and DS 1.3 is just to helpful to ignore it.
Signed-off-by: David Graeff david.graeff@web.de