-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Adds new integration [amitfin/patch] #2103
Conversation
7f8d339
to
70f49c8
Compare
I'm not going to allow this.
Not only will this be used to override core integrations, it will also be done persistently without any possibility of detection, or warning. This will also bypass safe mode (which is designed to start core without any custom modifications). For libraries I suggest https://developers.home-assistant.io/docs/creating_integration_manifest?_highlight=deps#custom-requirements-during-development--testing instead, as for core integration modifications you want to try, you can use custom components. |
Ok, fair enough. Just for the records, this integration was developed for solving a real blocker:
Anyway, as mentioned above, there is no real need to add the integration to HACS. It's still possible to use HACS as the installer via custom repository. |
BTW, I'll be glad to learn if there is a better solution for this blocker... |
The link to the developer docs I shared can be used for that. |
This is a production environment of HAOS on RPI4. |
I do not think so, but should not be needed. You may need to install it to the root of your config instead of in the deps directory inside it. |
Checklist
Links
Link to current release: https://github.com/amitfin/patch/releases/tag/v1.1.1
Link to successful HACS action (without the
ignore
key): https://github.com/amitfin/patch/actions/runs/6463495942/job/17546742448Link to successful hassfest action (if integration): https://github.com/amitfin/patch/actions/runs/6463495942/job/17546742580