[GRASS-git] [OSGeo/grass] 0fa154: CI: Reduce number of Python jobs (#3014)

Vaclav Petras noreply at github.com
Mon Jun 5 22:07:20 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: 0fa15493759ccc1116339625e4f1e3cbaf1616e5
      https://github.com/OSGeo/grass/commit/0fa15493759ccc1116339625e4f1e3cbaf1616e5
  Author: Vaclav Petras <wenzeslaus at gmail.com>
  Date:   2023-06-06 (Tue, 06 Jun 2023)

  Changed paths:
    R .github/workflows/black.yml
    R .github/workflows/flake8.yml
    R .github/workflows/pylint.yml
    A .github/workflows/python-code-quality.yml

  Log Message:
  -----------
  CI: Reduce number of Python jobs (#3014)

With increased use of pre-commit, having nice feedback from the CI checks is less important because contributors should get the feedback from pre-commit and CI is just to check that the files are correct, but not primarily to report to the contributor what quality controls did not pass because that's what should happen locally when the contributor follows contributing guidelines.

This applies to Black and Flake8. Pylint is not in pre-commit. Assuming the other get checked by pre-commit, the Pylint step is the one to fail most likely at the end of the job in case of issues.

Black and Flake8 checks (which are purely static) are done before compilation so that generated files are not checked.

Uses matrix with only single configuration to configure versions. Reports all versions of software in one place.




More information about the grass-commit mailing list