sampsyo / hass-smartthinq

Home Assistant component for LG SmartThinQ HVAC devices
MIT License
282 stars 98 forks source link

HACS Support #36

Closed brianhanifin closed 4 years ago

brianhanifin commented 4 years ago

I started over and am submitting the repository in a cleaner form. The only files I touched are: README.md, hacs.json, and info.md. None of the other files ended up needing to move.

hacs-screenshot

alexciesielski commented 4 years ago

Looking forward to getting this merged

sampsyo commented 4 years ago

This seems great overall! TBH I don't entirely understand the difference between info.md and our README we already have… can anyone who uses HACS clarify? In any case, it seems like the actual metadata is ready to go.

impankratov commented 4 years ago

@sampsyo regarding info.md

If you want to add a richer experience for your users you can add an info.md file to the root of your repository (this is optional, if you do not have this you need to set render_readme in the hacs.json file), this file will be rendered under the repository description, it does not support the full styling options as Github so use with care

Taken from https://hacs.xyz/docs/publish/start

sampsyo commented 4 years ago

OK, merged! That's all we need to do, right? We don't need to enter something into some database somewhere?

impankratov commented 4 years ago

@sampsyo

It could be added to HACS manually (I believe there's an option to import a repo.

But it also could be added to default HACS repository, so it could be available in HACS by default. More info here: https://hacs.xyz/docs/publish/include

javawizard commented 3 years ago

@sampsyo To be clear, you'd need to submit a PR against https://github.com/hacs/default/blob/master/integration adding this repo to the list in order for it to be included. They seem to like maintainers to be the ones to open the PR - is that something you'd have time to do?

sampsyo commented 3 years ago

Thanks for clarifying, @javawizard—that seems easy enough! I'll give it a try right now.

sampsyo commented 3 years ago

@javawizard I opened a PR but it looks like there's more subtlety to it. If you have time to help shepherd the PR through, we'd be grateful.

javawizard commented 3 years ago

@sampsyo Happy to help. What needs to be done? If this is re https://github.com/hacs/default/pull/550#issuecomment-662952965, then per https://github.com/hacs/default/pull/550#issuecomment-662956327 it sounds like that's no longer an issue.

sampsyo commented 3 years ago

Indeed! Sounds like that thing's OK. I think the only thing I'm asking for your help with (awesome, thanks for volunteering, by the way) is monitoring the thread to see if there's anything else we need to do. 🙏✨✨

javawizard commented 3 years ago

Sure, happy to keep an eye on it 👍

KTibow commented 3 years ago

As for brands, you can just duplicate custom_integrations/smartthinq_sensors to custom_integrations/smartthinq.