feat: improve incoming data processing performance by bdraco · Pull Request #1194 · python-zeroconf/python-zeroconf
Navigation Menu
{{ message }}
python-zeroconf / python-zeroconf Public
- Notifications You must be signed in to change notification settings
- Fork 229
Conversation
Copy link Copy Markdown
Member
bdraco
commented
Jul 8, 2023
bdraco
commented
- Make a few more classes that do not need dynamic attributes slotted
- Make a few more classes that do not need dynamic attributes slotted
Copy link Copy Markdown
Codecov Report
Patch coverage: 100.00% and no project coverage change.
Comparison is base (
f0577f0) 99.77% compared to head (6b2c3ba) 99.78%.
Additional details and impacted files
@@ Coverage Diff @@ ## master #1194 +/- ## ======================================= Coverage 99.77% 99.78% ======================================= Files 22 22 Lines 2721 2728 +7 Branches 474 474 ======================================= + Hits 2715 2722 +7 Misses 3 3 Partials 3 3
| Impacted Files | Coverage Δ | |
|---|---|---|
| src/zeroconf/__init__.py | 100.00% <100.00%> (ø) |
|
| src/zeroconf/_core.py | 99.77% <100.00%> (+<0.01%) |
⬆️ |
| src/zeroconf/_services/__init__.py | 100.00% <100.00%> (ø) |
|
| src/zeroconf/_services/browser.py | 99.14% <100.00%> (+0.01%) |
⬆️ |
| src/zeroconf/_services/registry.py | 100.00% <100.00%> (ø) |
☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.
bdraco commented Jul 8, 2023
src/zeroconf/_logger.py
Outdated
Show resolved
Hide resolved
src/zeroconf/_logger.py Outdated Show resolved Hide resolved
bdraco
marked this pull request as ready for review
bdraco
merged commit
a56c776
into
master
bdraco
deleted the
slots_2
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment