potiuk merged PR #36480:
URL: https://github.com/apache/airflow/pull/36480
--
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-unsubscr...@airflow.a
potiuk commented on PR #36480:
URL: https://github.com/apache/airflow/pull/36480#issuecomment-1872382584
Started discussion about it in
https://lists.apache.org/thread/fgoqj23hpl9mlo1n6gfkgvzjxvhfchzm
--
This is an automated message from the Apache Git Service.
To respond to the message,
potiuk commented on PR #36480:
URL: https://github.com/apache/airflow/pull/36480#issuecomment-1872373348
Actually ... that makes me think to propose to enable the
https://pre-commit.ci/ application for us - which should handle such issues by
automatically fixing such PRs.
I **thin
potiuk commented on PR #36480:
URL: https://github.com/apache/airflow/pull/36480#issuecomment-1872363062
> I have it installed; I just tried to sneak one past the tests directly
from the browser 😈 😅 . I won't be lazy moving forward.
Been there, done that :D
--
This is an automated
RNHTTR commented on PR #36480:
URL: https://github.com/apache/airflow/pull/36480#issuecomment-1872362628
> Some static checks. Highly recommend to have `pre-commit` installed :)
I have it installed; I just tried to sneak one past the tests directly from
the browser 😈 😅
--
This is
potiuk commented on PR #36480:
URL: https://github.com/apache/airflow/pull/36480#issuecomment-1871867355
Some static checks. Highly recommend to have `pre-commit` installed :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
RNHTTR opened a new pull request, #36480:
URL: https://github.com/apache/airflow/pull/36480
Based on trying to solve [this stack overflow
question](https://stackoverflow.com/questions/77730116/branching-not-working-in-airflow-as-expected/77730300#77730300),
it seems impossible to reliably b