search

Skipperro / pajgps-homeassistant

This integration allows you to use PAJ GPS devices (www.paj-gps.de) in Home Assistant.
MIT License
6 stars 0 forks source link

Pls. add repository.yaml to repo #2

Closed Miyamoto72 closed 7 months ago

Miyamoto72 commented 7 months ago

Hi! According to https://developers.home-assistant.io/docs/add-ons/repository/ all repos need to have a file repository.yaml in their root in order to be considered a valid repository.

repository.yaml needs to contain the following:

name: Name of repository
url: http://www.example/addons
maintainer: HomeAssistant Team <info@home-assistant.io>

You can find an example addon repo at https://github.com/home-assistant/addons-example which is a bit better documented.

If that file is found and valid it will be much easier to add the repo (and your addon) to the HA installation. So - please create one ;-)

Skipperro commented 7 months ago

Thanks for the info, I wasn't aware of that - I just added hacs.json to support installation through HACS. I'll add the file soon.

Skipperro commented 7 months ago

@Miyamoto72 Hey, correct me if I'm wrong, but I've looked into this topic and I found out, that repository.yaml file is a crucial part of Home Assistant Add-Ons, but it doesn't apply to Integrations like this Paj GPS component.

I've searched the GitHub for other custom integrations and none of them had the repository.yaml file. Even popular integrations, like integration with Amazon Alexa are featuring just hacs.json file: https://github.com/alandtse/alexa_media_player

Since this is custom integration, not Add-On, I don't see a reason for adding repository.yaml file. Am I wrong?

Miyamoto72 commented 7 months ago

Hi!

Oh - maybe I was wrong, 'cause I'm rather new to HomeAssistant and didn't get the difference betweeen Add-ons and Integrations. So - feel free to close this if you feel it would be more appropriate.

Am 2024-02-19 16:05, schrieb Skipperro:

@Miyamoto72 [1] Hey, correct me if I'm wrong, but I've looked into this topic and I found out, that repository.yaml file is a crucial part of Home Assistant Add-Ons, but it doesn't apply to Integrations like this Paj GPS component.

I've searched the GitHub for other custom integrations and none of them had the repository.yaml file. Even popular integrations, like integration with Amazon Alexa are featuring just hacs.json file: https://github.com/alandtse/alexa_media_player

Since this is custom integration, not Add-On, I don't see a reason for adding repository.yaml file. Am I wrong?

-- Reply to this email directly, view it on GitHub [2], or unsubscribe [3]. You are receiving this because you were mentioned.Message ID: @.***>

-- Die Zukunft hat viele Namen:

Für die Schwachen ist sie das Unerreichbare. Für die Furchtsamen ist sie das Unbekannte. Für die Tapferen ist sie die Chance.

(Victor Hugo)

Links:

[1] https://github.com/Miyamoto72 [2] https://github.com/Skipperro/pajgps-homeassistant/issues/2#issuecomment-1952640927 [3] https://github.com/notifications/unsubscribe-auth/AKK75555LOPJRXBKE4WLMJLYUNS2BAVCNFSM6AAAAABDOM3HUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJSGY2DAOJSG4 --=_d5ae6a336889e4626acd0d4a8bbdc6f7 Content-Type: multipart/related; boundary="=_b44e8771c78bd117e5772dd5ce0dbc8e"

--=_b44e8771c78bd117e5772dd5ce0dbc8e Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=UTF-8

Hi!

Oh - maybe I was wrong, 'cause I'm rather new to HomeAssistant and didn'= t get the difference betweeen Add-ons and Integrations. So - feel free to c= lose this if you feel it would be more appropriate.

Am 2024-02-19 16:05, schrieb Skipperro:


--

Die Zukunft hat viele Namen:

Für die Schwachen ist sie das Unerreichbare.
Für die Fur= chtsamen ist sie das Unbekannte.
Für die Tapferen ist sie die Ch= ance.

(Victor Hugo)

--=_b44e8771c78bd117e5772dd5ce0dbc8e Content-Transfer-Encoding: base64 Content-ID: @.***> Content-Type: image/gif; name=blocked.gif Content-Disposition: inline; filename=blocked.gif; size=118

R0lGODlhZAAyAIAAAPrOzgAAACH5BAAAAAAALAAAAABkADIAAAJNhI+py+0Po5y02ouz3rz7D4bi SJbmiabqyrbuC8fyTNf2jef6zvf+DwwKh8Si8YhMKpfMpvMJjUqn1Kr1is1qt9yu9wsOi8fksvls KwAAOw== --=_b44e8771c78bd117e5772dd5ce0dbc8e--

--=_d5ae6a336889e4626acd0d4a8bbdc6f7--

Skipperro commented 7 months ago

OK, closing this issue because the requested file is not required for custom integrations.