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.

Sign up for GitHub

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

Jump to bottom

Fix pycln undecidable cases #2051

Merged
merged 5 commits into from
Jul 24, 2023
Merged

Fix pycln undecidable cases #2051

merged 5 commits into from
Jul 24, 2023

Conversation

Avasam
Copy link
Collaborator

@Avasam Avasam commented Apr 21, 2023 

Pycln can not decide whether the unused imported names
are useless or imported to be used somewhere else (exported).

@Avasam
Copy link
Collaborator Author

Avasam commented Jul 8, 2023

@vernondcole Could I get a review on this?

@Avasam Avasam requested a review from mhammond July 23, 2023 17:40
Avasam
Avasam commented Jul 23, 2023
View reviewed changes
adodbapi/__init__.py Outdated Show resolved Hide resolved
@mhammond mhammond merged commit 630ffa3 into mhammond:main Jul 24, 2023
16 checks passed
@Avasam Avasam deleted the pycln-undecidable-cases branch July 24, 2023 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhammond mhammond mhammond approved these changes

@vernondcole vernondcole Awaiting requested review from vernondcole

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Avasam @mhammond @vernondcole