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

Python3.13 compat #809

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Nowa-Ammerlaan
Copy link

@Nowa-Ammerlaan Nowa-Ammerlaan commented Jan 2, 2025

Description

  • python 3.13 removed collections/abc.py, update the deprecated_aliases variable for this case
  • python 3.13 raises PermissionError earlier on initialization of the iterdir() instead of just on next()

Fixes #801

instead use _collections_abc for the alias

Bug: python-rope#801
Signed-off-by: Nowa Ammerlaan <[email protected]>
The PermissionError is now raised earlier when we initialize the iterdir
object instead of when we attempt to get the next()

Closes: python-rope#801
Signed-off-by: Nowa Ammerlaan <[email protected]>
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.

Test failures with Python 3.13.3rc3
1 participant