autopkgtest-cloud

Code review comment for ~paride/autopkgtest-cloud:pre-commit-pylint

  1. Git
  2. lp:~paride/autopkgtest-cloud
  3. pre-commit-pylint
  4. Merge into master
Revision history for this message
Paride Legovini (paride) wrote :

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)?

« Back to merge proposal