potiuk commented on code in PR #43185:
URL: https://github.com/apache/airflow/pull/43185#discussion_r1807340364
##########
.github/workflows/ci.yml:
##########
@@ -644,6 +644,24 @@ jobs:
( needs.build-info.outputs.run-kubernetes-tests == 'true' ||
needs.build-info.outputs.needs-helm-tests == 'true')
+ tests-task-sdk:
+ name: "Task SDK tests"
+ uses: ./.github/workflows/task-sdk-tests.yml
+ needs: [build-info, wait-for-ci-images]
+ permissions:
+ contents: read
+ packages: read
+ secrets: inherit
+ with:
+ runs-on-as-json-default: ${{
needs.build-info.outputs.runs-on-as-json-default }}
+ image-tag: ${{ needs.build-info.outputs.image-tag }}
+ default-python-version: ${{
needs.build-info.outputs.default-python-version }}
+ python-versions: ${{ needs.build-info.outputs.python-versions }}
+ debug-resources: ${{ needs.build-info.outputs.debug-resources }}
+ if: >
Review Comment:
This is a very bad design from GH -> because you can see that the workflow
is invalid only after you click on the summary page (so when you merge PRs you
also have to see if a number of workflows is > 10 :(
I raised it like 2 years ago to GH... Not fixed till today. Such PR should
have "status=failed" but it does not.
--
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]
