Skip to content

docs: fix the nonexistent IsDirectoryError name in a comment - #11068

Open
simpleqt wants to merge 1 commit into
python-poetry:mainfrom
simpleqt:sq918/isadirectory-comment
Open

simpleqt wants to merge 1 commit into
python-poetry:mainfrom
simpleqt:sq918/isadirectory-comment

Conversation

@simpleqt

Copy link
Copy Markdown
Contributor

src/poetry/installation/uninstaller.py:248 cites a Python exception IsDirectoryError that does not exist — the builtin is IsADirectoryError (which is exactly what the except two lines above catches). Comment-only fix; also fixes the article (aan).

The builtin is IsADirectoryError (what the except above catches);
IsDirectoryError does not exist.
Copilot AI lite review requested due to automatic review settings September 17, 2026 16:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

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