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

Switch to using aiofiles.open() to avoid warning, and replace deprecated aliases #380

Merged
merged 5 commits into from
Aug 27, 2024

Conversation

jmason
Copy link
Collaborator

@jmason jmason commented Jul 9, 2024

Thanks to @Dino0005, this resolves #377 -- in a better way than my first attempt. Also, I think his change was correct, the class for the integration is more correctly "iot_polling" instead of "iot_push"?

It also resolves #379 and #369 .

@jmason jmason marked this pull request as ready for review July 9, 2024 21:56
@jmason jmason marked this pull request as draft July 9, 2024 22:22
@jmason jmason changed the title Switch to using aiofiles.open() to avoid warning Switch to using aiofiles.open() to avoid warning, and replace deprecated aliases Jul 11, 2024
@jmason
Copy link
Collaborator Author

jmason commented Aug 21, 2024

@hultenvp I don't know if you are back yet, but this might be worth applying if you are happy with it?

@hultenvp hultenvp self-assigned this Aug 27, 2024
@hultenvp hultenvp merged commit 6d8f720 into hultenvp:master Aug 27, 2024
1 check passed
@hultenvp
Copy link
Owner

@jmason I'm back ;-)

Closes #377 , #379

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