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

[loxone] Removed Guava dependency #4111

Merged
merged 1 commit into from
Jan 23, 2019

Conversation

cweitkamp
Copy link
Contributor

  • Removed Guava dependency

Signed-off-by: Christoph Weitkamp [email protected]

Signed-off-by: Christoph Weitkamp <[email protected]>
@wborn wborn added the awaiting other PR Depends on another PR label Oct 8, 2018
@wborn
Copy link
Member

wborn commented Oct 8, 2018

Let's wait for #2905, #3973.

@cweitkamp
Copy link
Contributor Author

@wborn as the license updates created conflicts anyways should we think about merging this PR? wdyt?

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.

LGTM. It should be easy to resolve this minor change.

@wborn wborn added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Jan 23, 2019
@martinvw martinvw merged commit 2564a78 into openhab:master Jan 23, 2019
@cweitkamp cweitkamp deleted the feature-loxone-remove-guava branch January 23, 2019 13:32
@cweitkamp cweitkamp removed the awaiting other PR Depends on another PR label Jan 23, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Jun 19, 2019
Signed-off-by: Christoph Weitkamp <[email protected]>
Signed-off-by: Pshatsillo <[email protected]>
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
Signed-off-by: Christoph Weitkamp <[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