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 [Sese-Schneider/ha-cover-time-based] #2088

Merged
merged 4 commits into from
Jan 7, 2024

Conversation

Sese-Schneider
Copy link
Contributor

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/Sese-Schneider/ha-cover-time-based/releases/tag/v2.0.0
Link to successful HACS action (without the ignore key): https://github.com/Sese-Schneider/ha-cover-time-based/actions/runs/6430678535
Link to successful hassfest action (if integration): https://github.com/Sese-Schneider/ha-cover-time-based/actions/runs/6425890011

@hacs-bot hacs-bot bot changed the title Adds integration [Sese-Schneider/ha-cover-time-based] Adds new integration [Sese-Schneider/ha-cover-time-based] Oct 6, 2023
@@ -806,6 +806,7 @@
"Sennevds/media_player.template",
"sergeylysov/sst_cloud",
"sermayoral/ha-samsungtv-encrypted",
"Sese-Schneider/ha-cover-time-based",
Copy link
Member

Choose a reason for hiding this comment

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

The current release is not compatible.

Copy link
Contributor Author

@Sese-Schneider Sese-Schneider Jan 5, 2024

Choose a reason for hiding this comment

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

I tried to figure out what you exactly mean by "not compatible", but didn't understand the issue.
Please help me understand what I need to do to make it compatible @ludeeus

Things I've checked and done:

  • The changed LOC is at the correct place
  • hassfest workflow passes link
  • HACS workflow passes link
  • Integration works in production with 2024.01 screenshot
  • I've rechecked the publishing documentation if I did everything here
  • I've added a brand link

Copy link
Member

@ludeeus ludeeus Jan 5, 2024

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ludeeus thank you for your assistance!

The issue was that for an unknown reason the release v1.0.0 was still marked as the latest release in the GitHub API, while the GitHub UI was correctly updated with the release v2.0.0.
I don't know why that has happened, might have been an issue with GitHub, but I can't tell ...
Anyways, this lead me to believe everything was fine (the tests in my repo did pass after all and v2.0.0 was available) while in this repo the wrong version was fetched and the tests failed.

I've republished the release v2.0.0 and pushed a new one v2.0.1 for good measure, which is the latest release now.
Everything should work now 🤞🏼 HACS hassfest

I apologize for the inconvenience

Copy link
Member

Choose a reason for hiding this comment

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

Ah, i see 👍
I'll take another look at this repo the upcoming Sunday in that case 👍

@ludeeus ludeeus marked this pull request as draft December 24, 2023 20:47
@Sese-Schneider Sese-Schneider marked this pull request as ready for review January 5, 2024 10:17
@Sese-Schneider Sese-Schneider requested a review from ludeeus January 5, 2024 10:17
@Sese-Schneider
Copy link
Contributor Author

Sorry about re-requesting the review, I forgot this is not allowed here and I'm used to pressing the button from work ...

@ludeeus ludeeus merged commit c771a02 into hacs:master Jan 7, 2024
7 checks passed
@hacs-bot
Copy link

hacs-bot bot commented Jan 7, 2024

Your repository is now added to HACS 🎉
Here is a few resources that can be useful:

@Sese-Schneider Sese-Schneider deleted the patch-1 branch January 8, 2024 07:36
bramstroker pushed a commit to bramstroker/default that referenced this pull request Dec 21, 2024
Adds integration [Sese-Schneider/ha-cover-time-based]

Co-authored-by: Joakim Sørensen <[email protected]>
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.

2 participants