Skip to content

Add XML command SetCleanSpeed#911

Merged
edenhaus merged 6 commits into
DeebotUniverse:devfrom
nanomad:feature/xml-fan-speed-commands
Apr 27, 2025
Merged

Add XML command SetCleanSpeed#911
edenhaus merged 6 commits into
DeebotUniverse:devfrom
nanomad:feature/xml-fan-speed-commands

Conversation

@nanomad

@nanomad nanomad commented Apr 20, 2025

Copy link
Copy Markdown
Contributor

Depends on #907

@codecov

codecov Bot commented Apr 20, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.57%. Comparing base (b24dc56) to head (80450d7).
Report is 3 commits behind head on dev.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #911      +/-   ##
==========================================
+ Coverage   92.50%   92.57%   +0.06%     
==========================================
  Files         122      122              
  Lines        4672     4671       -1     
  Branches      294      292       -2     
==========================================
+ Hits         4322     4324       +2     
+ Misses        288      285       -3     
  Partials       62       62              

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq

codspeed-hq Bot commented Apr 20, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #911 will not alter performance

Comparing nanomad:feature/xml-fan-speed-commands (80450d7) with dev (3d3a65d)

Summary

✅ 6 untouched benchmarks

@nanomad nanomad force-pushed the feature/xml-fan-speed-commands branch from e55b256 to 1fb19c2 Compare April 22, 2025 11:10
@nanomad nanomad marked this pull request as ready for review April 22, 2025 11:14

@edenhaus edenhaus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please solve the merge conflicts

@nanomad nanomad force-pushed the feature/xml-fan-speed-commands branch from 1fb19c2 to 99673dd Compare April 24, 2025 11:00
@nanomad

nanomad commented Apr 24, 2025

Copy link
Copy Markdown
Contributor Author

Please solve the merge conflicts

@edenhaus conflicts solved, the CI pipeline crashed and needs restarting

@nanomad nanomad requested a review from edenhaus April 24, 2025 11:11
@nanomad nanomad force-pushed the feature/xml-fan-speed-commands branch from 99673dd to d73dbe1 Compare April 24, 2025 18:34
Comment thread deebot_client/commands/xml/__init__.py Outdated
Comment thread tests/commands/xml/test_fan_speed.py
@nanomad nanomad force-pushed the feature/xml-fan-speed-commands branch from 799b90a to b39ebb1 Compare April 25, 2025 15:55
@nanomad nanomad requested a review from edenhaus April 25, 2025 15:59
@edenhaus edenhaus added the pr: new-feature PR, which adds a new feature label Apr 25, 2025
nanomad and others added 4 commits April 26, 2025 11:54
@nanomad nanomad force-pushed the feature/xml-fan-speed-commands branch from b39ebb1 to f28aade Compare April 26, 2025 09:55
@nanomad

nanomad commented Apr 26, 2025

Copy link
Copy Markdown
Contributor Author

@edenhaus Rebased

@edenhaus edenhaus changed the title Implement XML fan speed commands Add XML command SetCleanSpeed Apr 27, 2025

@edenhaus edenhaus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @nanomad 👍

@edenhaus edenhaus enabled auto-merge (squash) April 27, 2025 15:30
@edenhaus edenhaus merged commit 8ed90f2 into DeebotUniverse:dev Apr 27, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: new-feature PR, which adds a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants