FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: brand new async examples, fresh readme example and DNSText repr by bdraco · Pull Request #1873 · python-zeroconf/python-zeroconf · GitHub

Repository navigation

chore: brand new async examples, fresh readme example and DNSText repr - #1873

Merged
bdraco merged 2 commits into
masterfrom
replace_inherited_display_blocks
Aug 29, 2026
Merged

bdraco merged 2 commits into
masterfrom
replace_inherited_display_blocks

Conversation

bdraco commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

Follow up to #1872 for #1835. The two async example files come back as brand new scripts (create_task and Event based, decoded_properties display, fresh structure); the readme gains a freshly written usage example using a context manager; and DNSText gets a new repr that reports the byte count for long payloads instead of the 2009 seven character prefix.

A four word comparison of the fresh files against the deleted ones shares only compelled machinery: the factual service type strings, imports, argparse boilerplate, and the required handler signature.

Test plan

  • suite passes; both examples run under --help and pass ruff

Copy link
Copy Markdown

Thanks for contributing! A quick note: this project is in the process of moving from LGPL-2.1 to Apache 2.0, see #1835.

By continuing with this pull request you are okay with your contribution, and your previous contributions to this repository, being offered under Apache 2.0.

If that does not work for you, no hard feelings; reply with the exact phrase I do not agree to the Apache 2.0 license or simply close this pull request, and nothing of yours will be included in the change.

codspeed Bot commented Aug 29, 2026 •
edited
Loading

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 24 untouched benchmarks


Comparing replace_inherited_display_blocks (b375a2e) with master (a595a5c)

codecov Bot commented Aug 29, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.87%. Comparing base (69fca98) to head (b375a2e).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1873   +/-   ##
=======================================
  Coverage   99.87%   99.87%           
=======================================
  Files          33       33           
  Lines        3878     3882    +4     
  Branches      566      567    +1     
=======================================
+ Hits         3873     3877    +4     
  Misses          3        3           
  Partials        2        2           

☔ View full report in Codecov by Harness.
📢 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.

Base automatically changed from remove_inherited_display_blocks to master August 29, 2026 22:21
bdraco force-pushed the replace_inherited_display_blocks branch from 117a60f to 0b1087d Compare August 29, 2026 22:21
bdraco marked this pull request as ready for review August 29, 2026 22:23
bdraco merged commit 644b18a into master Aug 29, 2026
33 of 34 checks passed
bdraco deleted the replace_inherited_display_blocks branch August 29, 2026 22:29
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL