Skip to content

Comments

Disable retries for requests, deferring to zigpy#264

Merged
puddly merged 2 commits intozigpy:devfrom
puddly:puddly/disable-retries
May 8, 2025
Merged

Disable retries for requests, deferring to zigpy#264
puddly merged 2 commits intozigpy:devfrom
puddly:puddly/disable-retries

Conversation

@puddly
Copy link
Contributor

@puddly puddly commented May 7, 2025

It looks like retries for this radio library were implemented a level deeper than ControllerApplication so they were missed when we moved them into zigpy.

@codecov
Copy link

codecov bot commented May 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.79%. Comparing base (70910bc) to head (8f668af).
Report is 3 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #264      +/-   ##
==========================================
- Coverage   98.80%   98.79%   -0.02%     
==========================================
  Files           7        7              
  Lines         920      911       -9     
==========================================
- Hits          909      900       -9     
  Misses         11       11              

☔ 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.

@puddly puddly merged commit 37241f1 into zigpy:dev May 8, 2025
14 checks passed
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.

1 participant