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

Declare connection on few bindings #14787

Merged
merged 6 commits into from
Apr 12, 2023
Merged

Declare connection on few bindings #14787

merged 6 commits into from
Apr 12, 2023

Conversation

hmerk
Copy link
Contributor

@hmerk hmerk commented Apr 11, 2023

Done for bindings I am maintaining.

Signed-off-by: Hans-Jörg Merk [email protected]

Related to #14885

@hmerk hmerk requested a review from jlaur as a code owner April 11, 2023 21:12
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.

Thanks!

@jlaur
Copy link
Contributor

jlaur commented Apr 12, 2023

@hmerk - I don't know why the DCO passed, but it seems like your commits are not signed off? I see this:
image

@hmerk
Copy link
Contributor Author

hmerk commented Apr 12, 2023

@hmerk - I don't know why the DCO passed, but it seems like your commits are not signed off? I see this: image

Cause I manually set it to pass after it did not when I created the PR via web editing.

@hmerk
Copy link
Contributor Author

hmerk commented Apr 12, 2023

CI build failure unrelated to this PR (failure reasoned by systeminfo tests).

@jlaur
Copy link
Contributor

jlaur commented Apr 12, 2023

Cause I manually set it to pass after it did not when I created the PR via web editing.

I'm not sure if there is a GitHub option for configuring a sign-off for the web editor, but otherwise you can manually add the sign-off message when preparing your commit. Manually approving the DCO without sign-off doesn't really solve the problem. 🙂

@hmerk
Copy link
Contributor Author

hmerk commented Apr 12, 2023

Manually approving the DCO without sign-off doesn't really solve the problem.

It does, as you can see, the DCO check is green.
I normally don't do web edits, but prefer proper PR's...
But will check how to set the options right.

EDIT: I think my Sign-off was just formatted incorrect.... Now it is corrected

@jlaur
Copy link
Contributor

jlaur commented Apr 12, 2023

Manually approving the DCO without sign-off doesn't really solve the problem.

It does, as you can see, the DCO check is green.

The requirement is to have properly signed commits, not green DCO check:
https://www.openhab.org/docs/developer/contributing.html#sign-your-work

In your case the commit message must contain "Signed-off-by: Hans-Jörg Merk [email protected]" which I found in a previously merged pull request by you.

For this PR I will manually copy that sign-off message.

@jlaur jlaur merged commit af09624 into openhab:main Apr 12, 2023
@jlaur jlaur added this to the 4.0 milestone Apr 12, 2023
FordPrfkt pushed a commit to FordPrfkt/openhab-addons that referenced this pull request Apr 20, 2023
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