ferruzzi opened a new pull request, #32358: URL: https://github.com/apache/airflow/pull/32358
Part of https://github.com/apache/airflow/issues/10742 D205 asserts that all docstrings must have a one-line summary ending in a period. If there is more than one sentence then there must be a blank line before the rest of the docstring. Meeting these requirements could be as simple as adding a newline, or might require some rephrasing. There are almost a thousand violations in the repo so we're going to have 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 contained in the following provider packages: - pagerduty - papermill - plexus - postgress - presto - qubole - redis - salesforce - samba - segment - sendgrid - sftp - singularity - slack - smtp ### To test If you comment out [this line](https://github.com/apache/airflow/blob/main/pyproject.toml#L68) and run pre-commit in main you will get 642 errors. After these changes, "only" 624 remain and no files in any of the above provider packages should be on the list. After uncommenting that line and rerunning 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]
