Skip to content

Update library for D225 support#1602

Open
alams154 wants to merge 3 commits intopython-kasa:masterfrom
alams154:feat/add-d225-library-support
Open

Update library for D225 support#1602
alams154 wants to merge 3 commits intopython-kasa:masterfrom
alams154:feat/add-d225-library-support

Conversation

@alams154
Copy link
Contributor

@alams154 alams154 commented Oct 29, 2025

Discovery fails because D225 does not provide an encrypt_info field in it's response json

@codecov
Copy link

codecov bot commented Oct 29, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.84%. Comparing base (29007e1) to head (1468f60).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1602      +/-   ##
==========================================
+ Coverage   92.82%   92.84%   +0.01%     
==========================================
  Files         157      157              
  Lines        9642     9654      +12     
  Branches      976      976              
==========================================
+ Hits         8950     8963      +13     
  Misses        492      492              
+ Partials      200      199       -1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alams154 alams154 mentioned this pull request Oct 29, 2025
Open
rytilahti
rytilahti requested changes Nov 5, 2025
View reviewed changes
Copy link
Member

@rytilahti rytilahti left a comment

Choose a reason for hiding this comment

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

Please clean up the unnecessary changes from this PR to make it easier to review, I will take a deeper look into the changes then.

@alams154 alams154 force-pushed the feat/add-d225-library-support branch 3 times, most recently from 811d469 to a5e6e97 Compare November 6, 2025 14:31
@alams154 alams154 force-pushed the feat/add-d225-library-support branch from a5e6e97 to 608be2c Compare November 6, 2025 14:37
@alams154 alams154 force-pushed the feat/add-d225-library-support branch from 8e4ddd1 to 1468f60 Compare November 6, 2025 15:16
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days.
If you are the author of this PR, please leave a comment if you want to keep it open. Also, please rebase your PR onto the latest dev branch to ensure that it's up to date with the latest changes.
Thank you for your contribution!

@github-actions github-actions bot added stale and removed stale labels Feb 5, 2026
@rytilahti rytilahti mentioned this pull request Feb 21, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rytilahti rytilahti rytilahti requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alams154 @rytilahti