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/01/31 07:36:07 UTC
[GitHub] [airflow] imamdigmi opened a new pull request #13995: Add DANA Indonesia to the Companies uses Airflow
imamdigmi opened a new pull request #13995:
URL: https://github.com/apache/airflow/pull/13995
Added DANA Indonesia to the Companies uses Airflow
@kaxil
----------------------------------------------------------------
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 #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343596
Ah yeah. looks good -static checks are green :)
----------------------------------------------------------------
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] imamdigmi commented on pull request #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
imamdigmi commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770342685
Ooops, sorry, i will fix it
----------------------------------------------------------------
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] imamdigmi commented on pull request #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
imamdigmi commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343795
Ooh yeah! great thanks for the feedback
----------------------------------------------------------------
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 edited a comment on pull request #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343505
> Hi @potiuk i updated the PR in this commit [07ee41d](https://github.com/apache/airflow/commit/07ee41d4b5630f50a17f610d48425d97699dcd23), is it the correct order?
> thanks!
FYI. Failed static checks will show you if it is wrong and will show you the right order - you just have to look at the output. And if you Install pre-commit, the order will be automatically corrected when you are trying to commit a wrong one.
----------------------------------------------------------------
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 #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770342125
Sorry - bad sorting order :)
----------------------------------------------------------------
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] imamdigmi commented on pull request #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
imamdigmi commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343199
Hi @potiuk i updated the PR in this commit 07ee41d4b5630f50a17f610d48425d97699dcd23, is it the correct order?
thanks!
----------------------------------------------------------------
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 #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #13995:
URL: https://github.com/apache/airflow/pull/13995
----------------------------------------------------------------
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] boring-cyborg[bot] commented on pull request #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343635
Awesome work, congrats on your first merged pull request!
----------------------------------------------------------------
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 #13995: Add DANA Indonesia to the Companies uses Airflow
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #13995:
URL: https://github.com/apache/airflow/pull/13995#issuecomment-770343505
> Hi @potiuk i updated the PR in this commit [07ee41d](https://github.com/apache/airflow/commit/07ee41d4b5630f50a17f610d48425d97699dcd23), is it the correct order?
> thanks!
Don't know. Failed static checks will show you if it is wrong and will show you the right order - you just have to look at the output. And if you Install pre-commit, the order will be automatically corrected when you are trying to commit a wrong one.
----------------------------------------------------------------
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