Skip to content

Conversation

@smcv
Copy link
Contributor

@smcv smcv commented Nov 11, 2024

Based on #154 to avoid merge conflicts, please review/merge that first.

I ran some QA tools against setup.py to check that I hadn't done anything obviously wrong in #154, and they produced some unrelated warnings, most of which are fixed here.

  • setup.py: Remove unused imports

    Detected by pyflakes3.

  • setup.py: Use PEP8 whitespace

    Detected by pycodestyle.

@multani
Copy link
Owner

multani commented Nov 11, 2024

Do you mind rebasing on the main branch (or merging from main)? I merged #154 and it now conflicts.
Thanks!

smcv added 2 commits November 11, 2024 19:24
Detected by pyflakes3.

Signed-off-by: Simon McVittie <[email protected]>
Detected by pycodestyle.

Signed-off-by: Simon McVittie <[email protected]>
@smcv smcv force-pushed the setup-py-cosmetic branch from da1064c to 984849b Compare November 11, 2024 19:25
@smcv
Copy link
Contributor Author

smcv commented Nov 11, 2024

Do you mind rebasing on the main branch (or merging from main)?

Sure, done.

@multani
Copy link
Owner

multani commented Nov 11, 2024

Great, thanks!

@multani multani merged commit a21468e into multani:master Nov 11, 2024
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