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 pathlib instead of py #4237

Merged
merged 1 commit into from
Mar 3, 2024
Merged

Use pathlib instead of py #4237

merged 1 commit into from
Mar 3, 2024

Conversation

Avasam
Copy link
Contributor

@Avasam Avasam commented Feb 26, 2024

Summary of changes

py was only installed as a dependency of pytest-fork which was only installed as a dependency of pytest-xdist. This also removes another untyped import for #2345 / #3979 / #4192

I could've bumped it to >=1.9.0 (first typed version: https://py.readthedocs.io/en/latest/changelog.html#id3) but I figured may as well just stop using it since: https://pypi.org/project/py/

NOTE: this library is in maintenance mode and should not be used in new code.
The py lib is a Python development support library featuring the following tools and modules:
py.path: uniform local and svn path objects -> please use pathlib/pathlib2 instead

Pull Request Checklist

@Avasam Avasam force-pushed the change-py-for-pathlib branch from 9cbc97b to 6d4b087 Compare February 26, 2024 02:05
@Avasam Avasam changed the title Change use pathlib instead of py Use pathlib instead of py Feb 26, 2024
@Avasam Avasam force-pushed the change-py-for-pathlib branch 2 times, most recently from 326cec2 to 3d0e213 Compare February 26, 2024 02:39
@Avasam Avasam force-pushed the change-py-for-pathlib branch from 3d0e213 to c6ebe3d Compare February 26, 2024 02:46
@Avasam Avasam closed this Feb 26, 2024
@Avasam Avasam reopened this Feb 26, 2024
@jaraco jaraco merged commit 2d5c7d5 into pypa:main Mar 3, 2024
27 of 42 checks passed
@Avasam Avasam deleted the change-py-for-pathlib branch March 3, 2024 16:15
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