← Back to team overview

canonical-ubuntu-qa team mailing list archive

Re: [Merge] ~paride/autopkgtest-cloud:pre-commit-pylint into autopkgtest-cloud:master

 

Individual pylint tags are tackled in separate commits.

Changes that need more attention in review:

* In CI I had to add a PPA where I backported pre-commit to Focal. That's under my LP user. Should I move that under canonical-ubuntu-qa?

* Any better way of installing Python dependencies than the yq trick I used in .launchpad.yaml?

* Do we like the two pylint hooks, one with --disable=import-error (meant for dev machines) and one without that flag (running in CI, where Python deps were installed)? 
-- 
https://code.launchpad.net/~paride/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/445630
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~paride/autopkgtest-cloud:pre-commit-pylint into autopkgtest-cloud:master.



References