The GitHub Actions job "Tests" on airflow.git has succeeded.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
82abd7991353ca4fb611129f76ba3f6095d7deb1 / Tzu-ping Chung <[email protected]>
Fix some non-critical pre-commit issues (#41749)

The '.venv' name (common for in-project virtual environments) is not
included in rat-excludes, causing a lint failure since a license is not
found in it. Since 'venv' (no leading dot) is excluded, let's also add
'.venv' to the file.

The 'downgradedb' method of BaseDBManager is declared abstract, but not
implemented in FABDBManager. This causes Mypy to error when FABDBManager
is instantiated. We don't always need the implement the method, so let's
remove the abstractmethod decorator.

Finally (and least critically), when you run pre-commit during a merge,
a hook checks if you're committing unresolved conflicts. The check is
not particularly smart, however, and can mis-identify header underlines
in restructuredText as false positives. This is easy to fix; just add
one more character to the underline. It's fine as long as the underline
is not EXACTLY SEVEN characters.

Report URL: https://github.com/apache/airflow/actions/runs/10579161723

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to