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

Enable installation via HACS #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Enable installation via HACS #15

wants to merge 4 commits into from

Conversation

ghjklw
Copy link

@ghjklw ghjklw commented Apr 26, 2021

Enable installation via HACS to make the process significantly easier and to enable easy updates and uninstall.

Changes

  • Reorganize the repository to follow the file structure required by HACS.
  • Update manifest.json to add all attributes required for integrations.
  • Add hacs.json manifest file.
  • Add Github actions to automatically validate every commit using HACS actions and Hassfest.
  • Update README.md to document installation through HACS.

Other changes to the repository/process required

  • To release a new version, the version number should be bumped in manifest.json and a new Github release should be created.
  • Github topics should be defined.

Further actions to publish to HACS repository

Relevant documentation

Malo JAFFRÉ added 3 commits April 26, 2021 17:14
Restructure the repository to implement HACS requirements.

Changes
=======
- Move the integration under the correct `custom_components repository`.
- Update manifest.json to include the required attributes for integrations.
- Create HACS manifest file - hacs.json.
- Update README.md to include HACS installation instructions.
Validate the repository against:
- HACS integrations requirements (https://github.com/hacs/action)
- Home Assistant's Hassfest validator (https://developers.home-assistant.io/blog/2020/04/16/hassfest/)

The `brands` validation is initially ignored as Grohe Sense is not yet added to
https://github.com/home-assistant/brands
@ghjklw
Copy link
Author

ghjklw commented Apr 26, 2021

If you accept this PR, I would obviously be happy to help with the next steps to get it added to HACS default repositories!

@seranoo
Copy link

seranoo commented May 1, 2021

👍👍👍

@BenTristem
Copy link

Thanks for working on this, I have emailed the repo owner today and hopefully they will start accepting PRs

@pertakman
Copy link

I made a pull request today with a suggestion to change from mbar to bar. Also suggested the version change in manifest.json without realizing that it was part of your pull request already.

Have you been able to get hold of Gunnar Kreitz?

@mattjgalloway
Copy link

Looks like this repo is probably stale now? I'd love HACS support for this though. Has anyone considered a fork yet and maintain it? I'd be happy to assist.

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.

5 participants