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

[knx] Made constants and handler packages internal #4048

Merged
merged 2 commits into from
Jan 26, 2019

Conversation

VelinYordanov
Copy link

See eclipse-archived/smarthome#4330

Signed-off-by: VelinYordanov [email protected]

wborn
wborn previously requested changes Sep 26, 2018
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 PR @VelinYordanov! Looks like the packages of two classes ended up wrong.

@wborn
Copy link
Member

wborn commented Sep 26, 2018

To reduce merge conflicts, we might want to merge #3500 first.

@wborn wborn added the awaiting other PR Depends on another PR label Sep 26, 2018
@wborn wborn dismissed their stale review September 30, 2018 11:11

Package corrected

@wborn wborn added the knx label Oct 26, 2018
Copy link
Contributor

@cweitkamp cweitkamp left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you both.

@cweitkamp cweitkamp merged commit 523fec3 into openhab:master Jan 26, 2019
@cweitkamp cweitkamp 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
* [knx] Made constants and handler packages internal

Also-by: Wouter Born <[email protected]>
Signed-off-by: VelinYordanov <[email protected]>
Signed-off-by: Pshatsillo <[email protected]>
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
* [knx] Made constants and handler packages internal

Also-by: Wouter Born <[email protected]>
Signed-off-by: VelinYordanov <[email protected]>
Signed-off-by: Maximilian Hess <[email protected]>
@wborn wborn added this to the 2.5 milestone Dec 16, 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