Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DNM: Test run: Rfc3986 urljoin optimize #1091

Closed
wants to merge 75 commits into from
Closed

Conversation

bdraco
Copy link
Member

@bdraco bdraco commented Sep 4, 2024

No description provided.

commonism and others added 30 commits September 2, 2024 07:53
urllib.parse.urljoin does not honor empty segments,
python/cpython#84774

implement using joinpath

change query string generation to not emit empty values as ""
always encode parameters with an assignment
adjust rfc3986 tests to include an assignment in the expectation
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
Co-authored-by: J. Nick Koston <[email protected]>
Co-authored-by: J. Nick Koston <[email protected]>
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Sep 4, 2024
Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.96%. Comparing base (fd25c6f) to head (1ced36b).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1091      +/-   ##
==========================================
+ Coverage   94.81%   94.96%   +0.14%     
==========================================
  Files          30       30              
  Lines        4358     4428      +70     
  Branches      382      390       +8     
==========================================
+ Hits         4132     4205      +73     
+ Misses        200      197       -3     
  Partials       26       26              
Flag Coverage Δ
CI-GHA 94.91% <100.00%> (+0.15%) ⬆️
MyPy 39.61% <82.05%> (+0.69%) ⬆️
OS-Linux 99.35% <100.00%> (+0.06%) ⬆️
OS-Windows 99.45% <100.00%> (+<0.01%) ⬆️
OS-macOS 99.03% <100.00%> (?)
Py-3.10.11 98.92% <100.00%> (-0.40%) ⬇️
Py-3.10.14 99.17% <100.00%> (+0.01%) ⬆️
Py-3.11.9 99.17% <100.00%> (+0.01%) ⬆️
Py-3.12.5 99.17% <100.00%> (+0.01%) ⬆️
Py-3.13.0-rc.1 99.17% <100.00%> (+0.01%) ⬆️
Py-3.8.10 98.86% <100.00%> (-0.39%) ⬇️
Py-3.8.18 99.12% <100.00%> (+0.05%) ⬆️
Py-3.9.13 98.86% <100.00%> (?)
Py-3.9.19 99.12% <100.00%> (+0.27%) ⬆️
Py-pypy7.3.11 99.17% <100.00%> (?)
Py-pypy7.3.16 99.20% <100.00%> (?)
VM-macos-latest 99.03% <100.00%> (?)
VM-ubuntu-latest 99.35% <100.00%> (+0.06%) ⬆️
VM-windows-latest 99.45% <100.00%> (+<0.01%) ⬆️
pytest 99.35% <100.00%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bdraco bdraco closed this Sep 4, 2024
@bdraco bdraco deleted the rfc3986-urljoin_optimize branch September 27, 2024 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants