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

Use a dedicated URL-handling library #1380

Closed
5 tasks done
jwodder opened this issue Dec 18, 2023 · 3 comments · Fixed by #1415
Closed
5 tasks done

Use a dedicated URL-handling library #1380

jwodder opened this issue Dec 18, 2023 · 3 comments · Fixed by #1415
Assignees
Labels
code quality Nonfunctional improvements to the codebase dependencies Update one or more dependencies version released

Comments

@jwodder
Copy link
Member

jwodder commented Dec 18, 2023

There are various places in the code that would benefit from the use of a modern URL-handling library, such as furl, purl, or yarl. These include:

We should use such a library to improve the code.

@jwodder jwodder added dependencies Update one or more dependencies version code quality Nonfunctional improvements to the codebase labels Dec 18, 2023
@yarikoptic
Copy link
Member

good idea. Only yarl seems to be alive, the other two had no changes in years. And dang -- yarl already has precommit setup for use of codespell ;-)

@yarikoptic
Copy link
Member

please proceed with making a decision on this one @jwodder to unblock

@jwodder jwodder self-assigned this Feb 29, 2024
yarikoptic added a commit that referenced this issue Mar 6, 2024
Use `yarl` to clean up some code
Copy link

🚀 Issue was released in 0.61.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Nonfunctional improvements to the codebase dependencies Update one or more dependencies version released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants