Skip to content

fix: restore sdist contents and listener typing after the reverts - #1839

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

bdraco merged 1 commit into
masterfrom
restore_after_revert

Conversation

@bdraco

@bdraco bdraco commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

Stacked on #1838; restores the two behaviors the reverts removed, written fresh.

Details

  • the sdist again includes COPYING, CHANGELOG.md, docs, and tests, expressed as a new include list
  • _respond_query in _listener.pxd is declared on a single line with the DNSIncoming typing current Cython requires
  • with this on top of revert: remove contributions from a non-consenting contributor #1838 the tree builds, all tests pass, and git blame shows no remaining lines from the non consenting contributor

Test plan

  • REQUIRE_CYTHON=1 build succeeds; full suite passes
  • poetry build sdist verified to contain COPYING, CHANGELOG.md, docs, and tests
  • blame audit over every tracked file returns zero lines from the reverted author

Base automatically changed from revert_nonconsented_contributions to master August 29, 2026 15:22
@bdraco
bdraco force-pushed the restore_after_revert branch from b7fe208 to 731160f Compare August 29, 2026 15:22
@bdraco
bdraco marked this pull request as ready for review August 29, 2026 15:22
@bdraco
bdraco force-pushed the restore_after_revert branch from 731160f to f491259 Compare August 29, 2026 15:22
@bdraco
bdraco merged commit 6b2a040 into master Aug 29, 2026
31 of 32 checks passed
@bdraco
bdraco deleted the restore_after_revert branch August 29, 2026 15:23
@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 (a353284) to head (f491259).
⚠️ Report is 1 commits behind head on master.

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

@codspeed

codspeed Bot commented Aug 29, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 24 untouched benchmarks


Comparing restore_after_revert (f491259) with master (6b2a040)1

Open in CodSpeed

Footnotes

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

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