Skip to content

Add tox workflow for Python CLI linting#581

Open
andmat900 wants to merge 4 commits intoeiffel-community:mainfrom
andmat900:20260331_enable_tox_workflow
Open

Add tox workflow for Python CLI linting#581
andmat900 wants to merge 4 commits intoeiffel-community:mainfrom
andmat900:20260331_enable_tox_workflow

Conversation

@andmat900
Copy link
Copy Markdown
Contributor

Applicable Issues

Fixes #495

Description of the Change

Add a GitHub Actions workflow that runs tox on the Python CLI code in the cli/ directory. This runs black, pylint, and pydocstyle checks on push to main and on pull requests, following the same pattern used in other ETOS repositories (e.g. etos-api, etos-library).

Also updates cli/tox.ini to remove the unused py3 test environment (no tests exist) and extends pylint and pydocstyle to cover all Python packages (etosctl and etos_debug in addition to etos_client).

Alternate Designs

Could have added tox as a job in the existing lint.yml workflow, but a separate workflow file is consistent with how other ETOS repositories handle Python linting.

Possible Drawbacks

The newly added etosctl and etos_debug packages may have existing lint issues that will now be flagged.

Sign-off

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

Signed-off-by: Andrei Matveyeu, andreimu@axis.com

Change-Id: Id1cf31f4f90d2f447739e94a8973cb97603f6a4b
Signed-off-by: Andrei Matveyeu <andreimu@axis.com>
@andmat900 andmat900 requested a review from a team as a code owner March 31, 2026 07:03
@andmat900 andmat900 requested review from fredjn and t-persson March 31, 2026 07:03
Change-Id: Ic9f0ee4c559a7d723f2bd26f89b8692a9dfd2c36
Change-Id: I9e97f603a28d8ef644a46012e0604c4752d52bb0
Change-Id: I360eb47a2d871a5c44abe054df2016af2131dde5
@andmat900 andmat900 requested a review from t-persson April 1, 2026 07:37
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.

Workflows that run tox are disabled in the ETOS repository

2 participants