Skip to content

wirego_remote(python): create initial pyproject + introduce formatter/linter#44

Merged
NothNoth merged 2 commits into
quarkslab:mainfrom
whiskeyo:wirego-remote-python-project-setup
Jun 10, 2025
Merged

wirego_remote(python): create initial pyproject + introduce formatter/linter#44
NothNoth merged 2 commits into
quarkslab:mainfrom
whiskeyo:wirego-remote-python-project-setup

Conversation

@whiskeyo
Copy link
Copy Markdown
Contributor

@whiskeyo whiskeyo commented May 20, 2025

  • Introduced pyproject.toml along with poetry to simplify working on the project.
  • Changed the project structure for easier management (that's the default setup for most of python projects).
  • Partially solves Add CI, formatter, linter & coding guidelines for all languages used #39 for Python. Reformatted & linted the whole code, fixing multiple errors where if not x in y was used (usually it meant if False in y) instead of if x not in y. Added CI for matrix of Python versions: 3.9, 3.10, 3.11, 3.12, 3.13.

@whiskeyo
Copy link
Copy Markdown
Contributor Author

whiskeyo commented Jun 2, 2025

Hi @NothNoth, what do you think about this change? Is it something worth going for in your opinion, or at this point it's not that important? I was considering adding some tests, but without a well-managed package it's a huge pain and it makes the job 100x harder.

@NothNoth
Copy link
Copy Markdown
Collaborator

NothNoth commented Jun 4, 2025

Hi !
I’ll take a look at it next week, I’m not at home currently

@NothNoth NothNoth merged commit b64ca3d into quarkslab:main Jun 10, 2025
10 checks passed
@NothNoth
Copy link
Copy Markdown
Collaborator

Just merged your PR, thanks a lot for your work!

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