You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/14 05:50:46 UTC

[GitHub] [airflow] potiuk commented on pull request #16964: [AIRFLOW 16730] Added docs & doc ref's for AWS transfer operators between SFTP & S3

potiuk commented on pull request #16964:
URL: https://github.com/apache/airflow/pull/16964#issuecomment-879610914


   Can you please @pyerbiz :
   1) rebase to latest main (there were an error in main fixed in #16971 
   2) fix static checks (the easiest way is to install pre-commit as suggested in the error message - it will then auto-correct everything for you.
   
   Having `pre-commit` installed for airflow contribution is HEARTILY recommended . It saves a lot of time on iterations and a lot of CI build time if you run the checks locally :)


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org