Remove holidays from manifest.json

jrfernandes / ontario_energy_board

Home Assistant component that installs a sensor with the current energy rate for Ontario energy companies

MIT License

41 stars 10 forks source link

Remove holidays from manifest.json #30

Closed deftdawg closed 11 months ago

deftdawg commented 1 year ago

Defer to the Home Assistant Core version of "holidays" instead

Resolves #28, #29

jrfernandes commented 11 months ago

holidays is a dependency. AFAIK, HASS core does not install this package. Certain integrations do require it (e.g. workday), making this to only work when one of those are installed.

For now I'll merge #28 to fix this issue.

deftdawg commented 11 months ago

Could one not add workday to "dependencies": [], to resolve the requirements without pinning versions & breaking each time they desync?