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

Remove redundant feature dependencies #16204

Merged
merged 2 commits into from Jan 5, 2024

Conversation

wborn
Copy link
Member

@wborn wborn commented Jan 4, 2024

The following features are already dependencies of openhab-runtime-base:

  • openhab-core-base
  • openhab-core-model-item
  • openhab-core-model-script
  • openhab-transport-mdns
  • openhab.tp-jackson
  • openhab.tp-jaxb

See also: #16202 (comment)

The following features are already dependencies of openhab-runtime-base:

* openhab-core-base
* openhab-core-model-item
* openhab-core-model-script
* openhab-transport-mdns

See also: openhab#16202 (comment)

Signed-off-by: Wouter Born <github@maindrain.net>
Copy link
Contributor

@andrewfg andrewfg left a comment

Choose a reason for hiding this comment

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

LGTM Hue

Copy link
Member

@GiviMAD GiviMAD left a comment

Choose a reason for hiding this comment

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

LGTM for androiddebugbridge and pulseaudio

…dencies

Signed-off-by: Wouter Born <github@maindrain.net>
@morph166955
Copy link
Contributor

Was the mdns added recently? I had issues when first building androidtv because I hadn't included it.

Copy link
Contributor

@Skinah Skinah left a comment

Choose a reason for hiding this comment

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

LGTM for ipcamera

@wborn
Copy link
Member Author

wborn commented Jan 5, 2024

Was the mdns added recently?

See #16202 (comment)

Copy link
Contributor

@stefan-hoehn stefan-hoehn left a comment

Choose a reason for hiding this comment

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

Nanoleaf looks good to me

Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

Thanks!

@J-N-K J-N-K added the enhancement An enhancement or new feature for an existing add-on label Jan 5, 2024
@J-N-K J-N-K added this to the 4.2 milestone Jan 5, 2024
@J-N-K J-N-K merged commit 4fe00bb into openhab:main Jan 5, 2024
3 checks passed
@wborn wborn deleted the remove-redundant-feature-deps branch January 5, 2024 19:05
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
* Remove redundant feature dependencies

The following features are already dependencies of openhab-runtime-base:

* openhab-core-base
* openhab-core-model-item
* openhab-core-model-script
* openhab-transport-mdns

See also: openhab#16202 (comment)

* Remove redundant openhab.tp-jackson and openhab.tp-jaxb feature dependencies

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants