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

Revert 4685 modbus internal #4704

Merged
merged 7 commits into from
Jan 25, 2019
Merged

Revert 4685 modbus internal #4704

merged 7 commits into from
Jan 25, 2019

Conversation

wborn
Copy link
Member

@wborn wborn commented Jan 25, 2019

Reverts #4685.

As @mrbig pointed out the exported packages are there so they can be used in https://github.com/openhab/openhab2-addons/pull/4220. Sorry for making them internal again!

We'd best review/merge https://github.com/openhab/openhab2-addons/pull/4220 so whenever we make them internal it results in build errors. 😉

@martinvw
Copy link
Member

Lets wait for the build :-)

@kaikreuzer
Copy link
Member

We'd best review/merge #4220 so whenever we make them internal it results in build errors. 😉

Go for it 😎!

@kaikreuzer
Copy link
Member

Lets wait for the build :-)

Travis should be enough, Jenkins has some memory issues atm...

@kaikreuzer kaikreuzer merged commit b1634eb into master Jan 25, 2019
@kaikreuzer kaikreuzer deleted the revert-4685-modbus-internal branch January 25, 2019 11:16
@mrbig
Copy link

mrbig commented Jan 25, 2019

Thank you @wborn for you understanding.
I see that you guys are doing a lot of work, so such things could happen.
Please keep up the good work you do for the community!

@wborn wborn added this to the 2.5 milestone Feb 28, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Jun 19, 2019
Signed-off-by: Pshatsillo <pshatsillo@gmail.com>
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
Signed-off-by: Maximilian Hess <mail@ne0h.de>
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.

None yet

4 participants