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

Sensor domain for energy sell/buy price #15

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

shiner66
Copy link
Contributor

Permits the selection of sensors to indicate energy buy and sell prices to use directly prices coming from energy integrations, which are exposed as sensors

Permits the selection of sensors to indicate energy buy and sell prices to use directly prices coming from energy integrations, which are exposed as sensors
@jmcollin78
Copy link
Owner

Thank you @shiner66 !

@jmcollin78
Copy link
Owner

Unfortunately CI failed. Can you please have a look ?

@shiner66
Copy link
Contributor Author

shiner66 commented Dec 13, 2023

Hello, it seems to be an issue independent from my PR

ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements_test.txt'

I tested the changes on my ha instance and works flawlessly

@jmcollin78
Copy link
Owner

Maybe something have change on the HA CI steps. Are you able to have a look on what happens ?

@shiner66
Copy link
Contributor Author

Maybe something have change on the HA CI steps. Are you able to have a look on what happens ?

Again, I have no clue, I believe the error is caused by the CI itself, you should investigate your actions code.
I don't know how my commit could in any way cause the following command to fail(?), I'm very confused haha

▶ Run python3 -m pip install -r requirements_test.txt

@jmcollin78 jmcollin78 merged commit 824b39d into jmcollin78:main Dec 13, 2023
3 of 4 checks passed
@jmcollin78
Copy link
Owner

Will see that later. For now I merge.

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.

2 participants