amoghrajesh commented on PR #57027: URL: https://github.com/apache/airflow/pull/57027#issuecomment-3441408948
@Srabasti the static check for trailing whitespaces are offloaded from the widely accepted and standard precommit checks defined in https://github.com/pre-commit/pre-commit-hooks, the errors will be clear to you if you run those hooks locally. The CI is meant to be an indicator that something is failing and it will be easily reproducible when you run it locally as well as auto fixed too. The hook also says that it modifies the files: ``` Remove trailing whitespace at end of line........................................Failed - hook id: trailing-whitespace - exit code: 1 - files were modified by this hook Fixing airflow-core/docs/start.rst ``` The static checks are fairly simple to install and use: https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst#using-prek and making edits from the UI for simple cases is easier but cases which have some changes like yours, would benefit better from using an IDE -- 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]
