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

[modbus][sunspec] Removed colons in Headline #7423

Merged

Conversation

Confectrician
Copy link
Contributor

Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

cc: @kaikreuzer

Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

Thanks - I indeed wondered why the website build fails!

Will have a closer look at this bundle as well as it is not a binding, but a sub part of one. Seems that it already caused the release notes to go wrong...

@kaikreuzer kaikreuzer merged commit 202bca7 into openhab:2.5.x Apr 19, 2020
@Confectrician Confectrician deleted the Confectrician-fix-website-build branch April 19, 2020 17:59
@J-N-K
Copy link
Member

J-N-K commented Apr 19, 2020

@kaikreuzer could it be related to the feature? For mqtt and bluetooth (where we have subbundles, too) we have an aggregated feature which installs all bundles at once. IIRC the feature is used for something during the build, right?

We decided against this aggregated feature here, but this could be changed (IMO the aggregated Features should be removed, I don‘t see why I have to install the homeassistant bundle if I don‘t use it, but want to use the generic MQTT channels).

@openhab openhab deleted a comment from TravisBuddy Apr 19, 2020
@kaikreuzer
Copy link
Member

IIRC the feature is used for something during the build, right?

The features define the installable units - we have one feature per add-on.
Add-on extensions are considered to be part of the add-on as we do not have any mechanism to treat them as an individual entity.

@J-N-K
Copy link
Member

J-N-K commented Apr 19, 2020

I can bring the feature in-line with the other ones tomorrow. But I still think this should be discussed for OH3. Especially for Bluetooth, where there are a lot of different bundles which probably will not be used all at the same time, the „always install everything“ strategy feels wrong.

@kaikreuzer
Copy link
Member

@J-N-K I have created #7424, which makes it in-sync with how extensions are done in the Bluetooth binding.

@cpmeister cpmeister added this to the 2.5.5 milestone Apr 20, 2020
yfre pushed a commit to yfre/openhab-addons that referenced this pull request Apr 27, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
Signed-off-by: Eugen Freiter <freiter@gmx.de>
@cpmeister cpmeister changed the title [Modbus][Sunspec] Removed colons in Headline [modbus][sunspec] Removed colons in Headline May 14, 2020
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request May 29, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
LoungeFlyZ pushed a commit to LoungeFlyZ/openhab2-addons that referenced this pull request Jun 8, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
J-N-K pushed a commit to J-N-K/openhab-addons that referenced this pull request Jul 14, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
CSchlipp pushed a commit to CSchlipp/openhab-addons that referenced this pull request Jul 26, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
Signed-off-by: CSchlipp <christian@schlipp.de>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
DaanMeijer pushed a commit to DaanMeijer/openhab-addons that referenced this pull request Sep 1, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
Signed-off-by: Daan Meijer <daan@studioseptember.nl>
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request Sep 19, 2020
Those seem to make the website build fail, since the yaml frontmatter is parsed wrong this way.

Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
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.

4 participants