ctc-oss / fapolicy-analyzer

Tools to assist with the configuration and management of fapolicyd.
https://ctc-oss.github.io/fapolicy-analyzer
GNU General Public License v3.0
12 stars 5 forks source link

Reformat Python code and add CI check #798

Closed jw3 closed 1 year ago

jw3 commented 1 year ago

Running black creates changes in about 40 files, which sort of prevents it from being used.

Goals

  1. Avoid wasting developer time formatting code
  2. Force developers to commit formatted code
  3. Enforce a uniform formatting of code across project

The PR to fix this should look like

  1. Run black to format all files and commit
  2. Add a CI check to the Python CI stage that will fail if there are any unformatted files in the commit

This doesn't need done until there is a lull in dev, ie. post v1.0, but should be done asap after that.