Add pyupgrade with pre-commit#297
Conversation
|
I love this PR, thanks for submitting it! Now that we are in python 3.10, moving away from the extra imports for things like list, dict, tuple, etc. is great. It looks like the extra imports are still at the top of each file though. Can you remove any unused imports that were created by this upgrade? |
|
@JBloss1517 I didn't want to go over 34 files manually, I was planning to automate it with ruff and isort. |
|
Got it. That makes sense. I'll let @JWock82 make the final call on merging this one. But I personally am on board with leaning into leveraging ruff for all of its built in capabilities like linting, import sorting, and formatting. This would consolidate several tools into one config and (very fast) tool. |
|
I did a bit more testing and I think this will yield to simplest & cleanest result: #299 I'll create another PR to add pyupgrade with ruff. |
|
Superseded by #300 |
Modernizes the codebase using pyupgrade. This was a very easy win on linting, but I can improve the setup by adding ruff and other common tools in following PRs.