Skip to content

💥 str(path) returns non protocol path for local filesystem#13

Merged
roy-ht merged 2 commits into
mainfrom
local-path-do-not-need-procotol-by-default
Aug 11, 2022
Merged

💥 str(path) returns non protocol path for local filesystem#13
roy-ht merged 2 commits into
mainfrom
local-path-do-not-need-procotol-by-default

Conversation

@roy-ht
Copy link
Copy Markdown
Owner

@roy-ht roy-ht commented Aug 11, 2022

No description provided.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #13 (7a8b587) into main (0daf346) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 7a8b587 differs from pull request most recent head 99db74b. Consider uploading reports for the commit 99db74b to get more accurate results

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   95.25%   95.28%   +0.02%     
==========================================
  Files           4        4              
  Lines         443      445       +2     
==========================================
+ Hits          422      424       +2     
  Misses         21       21              
Impacted Files Coverage Δ
src/pathlibfs/path.py 94.67% <100.00%> (+0.02%) ⬆️

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

@roy-ht roy-ht merged commit e6a6b66 into main Aug 11, 2022
@roy-ht roy-ht deleted the local-path-do-not-need-procotol-by-default branch August 11, 2022 23:45
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.

2 participants