-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix/linting #33
Fix/linting #33
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid you'll need to remove ops.testing.CAN_CONNECT
instead of ignoring the linting error. This is the default behaviour in ops 2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for tackling the linting and typing issues!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Test coverage for 94489c9
Static code analysis report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but it seems that trivy config was not found.
https://github.com/canonical/discourse-k8s-operator/actions/runs/4054111097/jobs/6975542344
Example of output in Indico:
"-a image" "-b table" "-c " "-d 1" "-e false" "-f os,library" "-g CRITICAL,HIGH" "-h " "-i ghcr.io/canonical/indico:4073362325" "-j ." "-k " "-l " "-m " "-n " "-o " "-p " "-q " "-r false" "-s " "-t " "-u " "-v trivy.yaml" "-z "
Running Trivy with trivy.yaml config from: trivy.yaml
It's part of a different PR, now everything is in place and I can update the chaos one that will have trivy, passing linters and jira action step. Thanks! |
This PR fixes linting issues found by mypy and pylint linters that are run via
tox -e lint
.