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

chore: record the jacobtomlinson consent in the ledger by bdraco · Pull Request #1920 · python-zeroconf/python-zeroconf · GitHub

Repository navigation

chore: record the jacobtomlinson consent in the ledger - #1920

Merged
bdraco merged 1 commit into
masterfrom
ledger_jacobtomlinson
Sep 28, 2026
Merged

bdraco merged 1 commit into
masterfrom
ledger_jacobtomlinson

Conversation

bdraco commented Sep 28, 2026

Copy link
Copy Markdown
Member

Adds the @jacobtomlinson consent received 2026-09-18 to RELICENSING.md, verified verbatim against the consent sentence, with the linked comment, date, and public commit address in the collapsed mapping. That makes 24 recorded consents with 18 contributors pending.

bdraco marked this pull request as ready for review September 28, 2026 14:12

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, and may keep LGPL-2.1-or-later as an OR option so GPL-2.0-only projects can keep using the library, see #1835.

By continuing with this pull request you agree that this new contribution is offered under both Apache 2.0 and LGPL-2.1-or-later, so it is covered under either outcome.

This notice does not affect your previous contributions. If you would like to consent to relicensing those as well, please post the consent statement on #1835; silence is never treated as consent for past work.

If offering this contribution under these licenses 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.

bdraco merged commit 27ee975 into master Sep 28, 2026
32 of 34 checks passed
bdraco deleted the ledger_jacobtomlinson branch September 28, 2026 14:12

codecov Bot commented Sep 28, 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 (bb91be5) to head (5ae4058).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1920   +/-   ##
=======================================
  Coverage   99.87%   99.87%           
=======================================
  Files          33       33           
  Lines        3911     3911           
  Branches      568      568           
=======================================
  Hits         3906     3906           
  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 Bot commented Sep 28, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 25.3%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 2 regressed benchmarks
✅ 22 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ test_hash_ipv4_address 1.1 ms 1.5 ms -25.31%
❌ test_hash_ipv6_address 1.1 ms 1.5 ms -25.3%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing ledger_jacobtomlinson (5ae4058) with master (bb91be5)

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