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 2022/10/12 15:19:00 UTC
[GitHub] [airflow] merobi-hub opened a new pull request, #27015: [Dev] fix bug in protm script
merobi-hub opened a new pull request, #27015:
URL: https://github.com/apache/airflow/pull/27015
The previous effort to add "#protm" tags to the PRotM script in dev/stats had an undefined variable in it that wasn't caught be the static code checks on CI.
This PR re-adds the previous changes and fixes the bug. The script now passes static code checks and works as expected when run locally.
Signed-off-by: Michael Robinson <me...@gmail.com>
--
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
[GitHub] [airflow] merobi-hub commented on pull request #27015: [Dev] fix bug in protm script
Posted by GitBox <gi...@apache.org>.
merobi-hub commented on PR #27015:
URL: https://github.com/apache/airflow/pull/27015#issuecomment-1276577760
Tests are running now.
> On Oct 12, 2022, at 12:48, Ephraim Anierobi ***@***.***> wrote:
>
>
> Can you rebase let see if the Build docs test would run
>
> —
> Reply to this email directly, view it on GitHub <https://github.com/apache/airflow/pull/27015#issuecomment-1276465686>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQKPOMZUGOIGOPAOWQ6R46TWC3TVVANCNFSM6AAAAAARDMWNGE>.
> You are receiving this because you authored the thread.
>
--
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
[GitHub] [airflow] ephraimbuddy merged pull request #27015: [Dev] fix bug in protm script
Posted by GitBox <gi...@apache.org>.
ephraimbuddy merged PR #27015:
URL: https://github.com/apache/airflow/pull/27015
--
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
[GitHub] [airflow] ephraimbuddy commented on pull request #27015: [Dev] fix bug in protm script
Posted by GitBox <gi...@apache.org>.
ephraimbuddy commented on PR #27015:
URL: https://github.com/apache/airflow/pull/27015#issuecomment-1276465686
Can you rebase let see if the Build docs test would run
--
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