Skip to content

revert: remove prose found by the fuzzy paraphrase audit - #1854

Merged
bdraco merged 1 commit into
masterfrom
remove_fuzzy_audit_prose
Aug 29, 2026
Merged

bdraco merged 1 commit into
masterfrom
remove_fuzzy_audit_prose

Conversation

@bdraco

@bdraco bdraco commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

Sixth removal pass for #1835. Every earlier audit required exact runs, so a reworded line escaped them; this pass used fuzzy line matching (token overlap plus sequence ratio) between every pyzeroconf era prose line and current master. It caught the production copies of the 2009 send docstring in _core.py, the verbatim unregister all services docstrings, the remaining tests equality family in _dns.py, the surviving first line of the add_additional_answer docstring, and derivative lines in _utils/time.py and _utils/name.py, plus two modern NSEC docstrings that copied the 2009 sentence templates, deleted for consistency. The RFC 6763 quotation went with its enclosing docstring and returns with the fresh one.

Deletion only, suite stays green. The follow up restores fresh text.

Test plan

  • full suite passes
  • fuzzy audit re-run reports only generic scenes a faire lines (unit tests for X, shebangs)

@bdraco
bdraco marked this pull request as ready for review August 29, 2026 20:07
@bdraco
bdraco merged commit 8afd02f into master Aug 29, 2026
33 of 34 checks passed
@bdraco
bdraco deleted the remove_fuzzy_audit_prose branch August 29, 2026 20:08
@github-actions

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

codspeed Bot commented Aug 29, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 24 untouched benchmarks


Comparing remove_fuzzy_audit_prose (c362c9e) with master (f01845a)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (4339329) during the generation of this report, so f01845a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

@codecov

codecov Bot commented Aug 29, 2026 •

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1854   +/-   ##
=======================================
  Coverage   99.87%   99.87%           
=======================================
  Files          33       33           
  Lines        3882     3882           
  Branches      567      567           
=======================================
  Hits         3877     3877           
  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.

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