Skip to content

Conversation

@Decodetalkers
Copy link

some part copy from url lib

solve: #32

@Decodetalkers Decodetalkers force-pushed the add_path_methods branch 2 times, most recently from 427cdf3 to e50b873 Compare April 15, 2025 14:45
@Decodetalkers
Copy link
Author

@yescallop will this be ok?

After reading the rfc and reading the code, I found this crate is more standing than that of mozilla, so I want to help finish this project. This is my first pr for this crate

@Decodetalkers Decodetalkers force-pushed the add_path_methods branch 5 times, most recently from dca8039 to 811d52e Compare April 16, 2025 12:43
@yescallop
Copy link
Owner

I appreciate your effort to contribute, but I haven't been able to work on the project and likely won't be able to for a while. It is still maintained though. I'll take a look when I get a chance.

@eirnym
Copy link
Contributor

eirnym commented Apr 27, 2025

LGTM

FYI: File paths on Windows with / file separator are accepted by the system API.

@Decodetalkers Decodetalkers force-pushed the add_path_methods branch 2 times, most recently from b159a34 to e4f455b Compare August 8, 2025 01:41
some part copy from url lib
@Decodetalkers
Copy link
Author

@yescallop Can you take a review? thanks

@yescallop
Copy link
Owner

I have not been able to gather my mind to consider this. Every time I think about it thoroughly it gives me a headache, to be honest. I will come back to this in the future.

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.

3 participants