EricGao888 edited a comment on pull request #21954:
URL: https://github.com/apache/airflow/pull/21954#issuecomment-1060374201


   @potiuk Hello, as the initial PR to add a new provider to airflow, this PR 
contains many new files. Should I add all the unit tests in order to merge this 
PR? If so, would it be better to separate this work into several sub PRs since 
there will be twice amount of new files once the unit tests are added. It will 
be difficult for reviewers to review such a pr. Any suggestions will be 
appreciated! : )


-- 
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]


Reply via email to