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

Adjust to core changes (addon.xml) #793

Merged
merged 1 commit into from
Jan 15, 2023
Merged

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Jan 15, 2023

ONLY merge after the corresponding code PR (openhab/openhab-core#3050) has been merged.

Signed-off-by: Jan N. Klug github@klug.nrw

Signed-off-by: Jan N. Klug <github@klug.nrw>
<name>ZigBee Binding</name>
<description>This is the binding for ZigBee providing a vendor neutral ZigBee implementation supporting many different
ZigBee devices.</description>
<author>Chris Jackson</author>
Copy link
Member

Choose a reason for hiding this comment

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

Following the new rule that we do not use this field for individual authors, shouldn't we rather remove this line, just as we did for all bindings in openhab-addons?

Copy link
Member Author

Choose a reason for hiding this comment

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

This rule is not new, we already had it before, I don‘t know why it was not adopted here, so I left it as-is.

Copy link
Member

Choose a reason for hiding this comment

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

The reason is simple - we simply forgot it, because it was in a different repository.
But you are right, it is not directly related to this PR, so I'll create a separate PR for it.

@kaikreuzer kaikreuzer merged commit 7a38e68 into openhab:main Jan 15, 2023
@J-N-K J-N-K deleted the addons-xml branch January 15, 2023 15:37
pgaufillet pushed a commit to pgaufillet/org.openhab.binding.zigbee that referenced this pull request Sep 7, 2023
Signed-off-by: Jan N. Klug <github@klug.nrw>
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.

2 participants