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 2020/12/31 16:48:34 UTC

[GitHub] [airflow] soltanianalytics opened a new pull request #13410: Improves documentation regarding providers and custom connections 2

soltanianalytics opened a new pull request #13410:
URL: https://github.com/apache/airflow/pull/13410


   See https://github.com/apache/airflow/pull/13375 for the first part
   
   The slack discussion went on and I added some more information that might be helpful for others as well. I also fixed a formatting bug that I introduced in the first part.


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

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

[GitHub] [airflow] soltanianalytics commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
soltanianalytics commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753361828


   Made some updates - let me know if there is any part you'd like me to go over again :)


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

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

[GitHub] [airflow] soltanianalytics commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
soltanianalytics commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753380873


   > Looks great! Merging! Thanks again ! What's next :)?
   
   Thanks! This is next: https://github.com/apache/airflow/pull/13408 ^^


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

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

[GitHub] [airflow] potiuk commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753372578


   Looks great! Merging! Thanks again ! What's next :)? 


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

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

[GitHub] [airflow] potiuk commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753018689


   Some spelling errors unfortunately :(


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

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

[GitHub] [airflow] potiuk commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753006912


   Good one! 
   
   Can you please rebase to latest master + solve conflicts ? 
   
   Happy New Year!


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

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

[GitHub] [airflow] potiuk merged pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #13410:
URL: https://github.com/apache/airflow/pull/13410


   


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

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

[GitHub] [airflow] soltanianalytics commented on pull request #13410: Improves documentation regarding providers and custom connections 2

Posted by GitBox <gi...@apache.org>.
soltanianalytics commented on pull request #13410:
URL: https://github.com/apache/airflow/pull/13410#issuecomment-753008750


   Done, thanks!
   
   Happy new year to you, too!


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

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