potiuk commented on PR #59567: URL: https://github.com/apache/airflow/pull/59567#issuecomment-3666194831
> While creating a small PR to the documentation, I ran into some difficulty with the pre-commit configuration. In particular, there were missing dependencies for the prek scripts. This PR aims to fix that issue. You should switch to `prek` instead. Requirements are stored in the scripts as "inline-metadata" https://packaging.python.org/en/latest/specifications/inline-script-metadata/ - and prek already supports them, while `pre-commit` does not. We do not want to duplicate those dependencies and keep `pre-commit` compatibility. Our contributing docs have been updated to reflect that - only `prek` is supported now. Also prekl supports workspaces in monorepo - and we are in progress of splitting our configuration files among subdirectories (we have now 11 .pre-commit-config.yaml files). This have been discussed and announced here: https://lists.apache.org/thread/dzs4s8wr8jd7q5r3lvcov2nhqpgt1031 Simply - switch to prek. -- 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]
