Skip to content

Add precommit bandit check - #106

Merged
ponceta merged 4 commits into
mainfrom
poa_add_bandit_precommit_check
Sep 15, 2026
Merged

ponceta merged 4 commits into
mainfrom
poa_add_bandit_precommit_check

Conversation

@ponceta

@ponceta ponceta commented Sep 15, 2026

Copy link
Copy Markdown
Member

General

  • Fix a bug
  • Add a feature
  • Maintenance / sustainability
  • Add Documentation

Describe your changes

  • Add precommit bandit check
  • Fix errors and apply best practices

Screenshots

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • CI Tests are green
  • The documentation is up to date with the proposed change.
  • My work is ready for review

Checklist before merge

  • A review has been performed
  • Comments are resolved
  • Documentation is ready

@ponceta ponceta self-assigned this Sep 15, 2026
@ponceta ponceta added the fix label Sep 15, 2026
@ponceta ponceta linked an issue Sep 15, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown

🔌 Plugin Artifact

A development build of the plugin is available for testing:

⬇️ Download teksi_module_management_tool_dev.zip

⚠️ This is a development build from commit 13cc4bf.

@ponceta

ponceta commented Sep 15, 2026

Copy link
Copy Markdown
Member Author

@3nids I have 80 bandit errors on the tests lib.

image

But none displayed here.

And additionally, any idea why python 14 is used and fails the precommit test when python 12 is specified as current python version?

@ponceta
ponceta merged commit 6c0e4c5 into main Sep 15, 2026
1 of 2 checks passed
@ponceta
ponceta deleted the poa_add_bandit_precommit_check branch September 15, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI/CD : Bandit tests are not passing on TMMT

2 participants