Skip to content

Fix truediv joining against absolute URL with no path#857

Merged
mjpieters merged 1 commit intomasterfrom
truediv_empty_path_fix
Apr 25, 2023
Merged

Fix truediv joining against absolute URL with no path#857
mjpieters merged 1 commit intomasterfrom
truediv_empty_path_fix

Conversation

@mjpieters
Copy link
Copy Markdown
Contributor

When joining path elements against an absolute URL, ensure that
the resulting path starts with a slash.

Fixes #854

When joining path elements against an absolute URL, ensure that
the resulting path starts with a slash.
@psf-chronographer psf-chronographer Bot added the bot:chronographer:provided There is a change note present in this PR label Apr 25, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2023

Codecov Report

Merging #857 (ebf326f) into master (ace3552) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #857   +/-   ##
=======================================
  Coverage   99.74%   99.74%           
=======================================
  Files           4        4           
  Lines         770      772    +2     
  Branches      218      219    +1     
=======================================
+ Hits          768      770    +2     
  Misses          2        2           
Flag Coverage Δ
unit 99.61% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
yarl/_url.py 99.66% <100.00%> (+<0.01%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mjpieters mjpieters merged commit 475ee4a into master Apr 25, 2023
@mjpieters mjpieters deleted the truediv_empty_path_fix branch April 25, 2023 15:48
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.

Regression: Equality of joined URLs fails to be compared equal with a readymade URL object.

1 participant