ferruzzi opened a new pull request, #33333: URL: https://github.com/apache/airflow/pull/33333
Part of https://github.com/apache/airflow/issues/10742 D401 asserts "First line should be in imperative mood". This is a little opaque so upon further investigation what they want is: ``` [Docstring] prescribes the function or method's effect as a command: ("Do this", "Return that"), not as a description; e.g. don't write "Returns the pathname ...". ``` There are over two thousand violations in the repo so I'm going to take this in bites. ### PLEASE NOTE There should be zero logic changes in this PR, only changes to docstrings and whitespace. If you see otherwise, please call it out. ### Included in this chunk All files located in the following modules: - airflow/api - airflow/api_connexion - airflow/api_internal - airflow/auth ### To test If you uncomment [this line](https://github.com/apache/airflow/blob/main/pyproject.toml#L61) and run pre-commit in main you will get literally thousands of errors. After these changes, no files in the list above should be on the list. After re-commenting that line and re-running pre-commits, there should be zero regressions. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
