Skip to content

CFE-4632: Added python linting in github actions#122

Open
victormlg wants to merge 4 commits intocfengine:masterfrom
victormlg:linting
Open

CFE-4632: Added python linting in github actions#122
victormlg wants to merge 4 commits intocfengine:masterfrom
victormlg:linting

Conversation

@victormlg
Copy link
Contributor

No description provided.

@victormlg victormlg force-pushed the linting branch 11 times, most recently from 5c9acec to 6db2a0c Compare February 13, 2026 15:17
@victormlg victormlg marked this pull request as ready for review February 13, 2026 15:18
@victormlg victormlg force-pushed the linting branch 5 times, most recently from 387dc68 to e7a9ed4 Compare February 16, 2026 09:26
Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sweet. thanks for the change to mostly in a script! Just one more nudge to move linting.sh to modules/ci directory. 👍

Signed-off-by: Victor Moene <victor.moene@northern.tech>
Signed-off-by: Victor Moene <victor.moene@northern.tech>
Signed-off-by: Victor Moene <victor.moene@northern.tech>
Fixed the way ansible promise type handldes missing ansible dependency to be compatible with pyright. Moreover, Ansible doesn't error anymore on unknown functionn init_plugin_loader() when ansible is not installed.

Signed-off-by: Victor Moene <victor.moene@northern.tech>
Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the linting and fixes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants