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

Apply labels from add-on build.(yaml|json) when building add-on #203

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

lildude
Copy link
Contributor

@lildude lildude commented Mar 28, 2024

I've discovered that the add-on builder doesn't apply the labels configured in the add-on's build.(yaml|json) file.

You can see this in action in the example add-on repo's packages as the "About this version" section hasn't been auto-populated:

Screenshot 2024-03-28 at 15 26 52

... and the manifest shows the labels haven't been applied either:

Screenshot 2024-03-28 at 15 27 12

This PR rectifies this by parsing the build.(yaml|json) file for the labels in build_addon() in the same way as the other build functions do.

lildude added a commit to lildude/ha-addon-ghostfolio that referenced this pull request Apr 7, 2024
lildude added a commit to lildude/ha-addon-ghostfolio that referenced this pull request Apr 7, 2024
* Bump ghostfolio/ghostfolio from 2.65.0 to 2.71.0

Bumps ghostfolio/ghostfolio from 2.65.0 to 2.71.0.

---
updated-dependencies:
- dependency-name: ghostfolio/ghostfolio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update versions

* Move labelling to dockerfile

Bug: home-assistant/builder#203

* Bump version

* Remove \

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Colin Seymour <colin@symr.io>
@agners agners merged commit fc1398e into home-assistant:master Apr 11, 2024
9 checks passed
@lildude lildude deleted the addon-labels branch April 11, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants