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

Adds new integration [BJReplay/ha-solcast-solar] #2535

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

Conversation

BJReplay
Copy link

@BJReplay BJReplay commented Jun 1, 2024

Checklist

  • I've read the publishing documentation.
  • I've added the HACS action to my repository.
  • (For integrations only) I've added the hassfest action to my repository.
  • The actions are passing without any disabled checks in my repository.
  • I've added a link to the action run on my repository below in the links section.
  • I've created a new release of the repository after the validation actions were run successfully.

Links

Link to current release: https://github.com/BJReplay/ha-solcast-solar/releases/tag/v4.1.7
Link to successful HACS action (without the ignore key): https://github.com/BJReplay/ha-solcast-solar/actions/runs/10859969691
Link to successful hassfest action (if integration): https://github.com/BJReplay/ha-solcast-solar/actions/runs/10859969724

This is a replacement for the integration mentioned in hacs/integration#3745 that has been deleted by the original author.

@hacs-bot hacs-bot bot changed the title Add BJReplay/ha-solcast-solar Adds new integration [BJReplay/ha-solcast-solar] Jun 1, 2024
Copy link

@lordvalium lordvalium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes okay

Copy link

@svisagie svisagie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good

Copy link

@EduardTheThird EduardTheThird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve of these changes.

@chronvas
Copy link

Hello @ludeeus 👋 can this be added? Thanks

@BJReplay
Copy link
Author

Hello @ludeeus 👋 can this be added? Thanks

Please don't nag the repo owner. As author of this PR, I am happy to wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants