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

Update pydantic to 2.x and use compatibility layer #386

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

Conversation

deejay1
Copy link

@deejay1 deejay1 commented Jan 10, 2025

Use v2's compatibility layer for a simple upgrade path for now.

Should do well enough until we figure out how to migrate to full v2 in #384 or other.

Use v2's compatibility layer for a simple upgrade path
for now.
@desosav
Copy link

desosav commented Jan 10, 2025

@deejay1, once again thank you for your efforts. :)
I am just letting you know that Ι have just tested this with HA 2025.1.2 and this is what I receive in the logs:

Unable to install package mytoyota @ git+https://github.com/deejay1/mytoyota.git@simpler_first: × No solution found when resolving dependencies: ╰─▶ Because mytoyota==0.0.0.post271.dev0+39a47e8 depends on pydantic>=2.10.5,<3.0.0 and pydantic==2.10.4, we can conclude that mytoyota==0.0.0.post271.dev0+39a47e8 cannot be used. And because only mytoyota==0.0.0.post271.dev0+39a47e8 is available and you require mytoyota, we can conclude that your requirements are unsatisfiable.

I hope it helps.

@deejay1
Copy link
Author

deejay1 commented Jan 10, 2025

@desosav I lowered the dependency a bit, see if it works now. If you could provide me with a fast way to test it with HA it would be great!

@desosav
Copy link

desosav commented Jan 10, 2025

@desosav I lowered the dependency a bit, see if it works now. If you could provide me with a fast way to test it with HA it would be great!

It seems OK now.
No more errors and the integration loads successfully.
Appreciated 👍

@maxim-yegorov
Copy link

Works for me as well
Thank you!

@skgsergio
Copy link
Contributor

I've tested it too and works, thanks.

@desosav I lowered the dependency a bit, see if it works now. If you could provide me with a fast way to test it with HA it would be great!

Regarding this... I've SSH'd into my HAOS machine and ran:

docker exec -it homeassistant uv pip install git+https://github.com/DurgNomis-drol/mytoyota.git@refs/pull/386/head

# also works with:
docker exec -it homeassistant uv pip install git+https://github.com/deejay1/mytoyota.git@simpler_first

Then just restarted HA and the integration started working.

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.

4 participants