sadpandajoe commented on code in PR #43975:
URL: https://github.com/apache/superset/pull/43975#discussion_r3953505927
##########
.github/workflows/superset-python-presto-hive.yml:
##########
@@ -80,13 +80,13 @@ jobs:
persist-credentials: false
submodules: recursive
- name: Setup Python
- uses: ./.github/actions/setup-backend/
+ uses: $/.github/actions/setup-backend/
- name: Setup Postgres
- uses: ./.github/actions/cached-dependencies
+ uses: $/.github/actions/cached-dependencies
Review Comment:
These references now resolve `cached-dependencies` through the main
repository tree, but that path is a submodule gitlink rather than a directory.
The self-repository resolver cannot load its `action.yml` from the submodule,
so both Presto/Hive jobs fail before tests start. Could these four references
remain workspace-relative after the recursive checkout, while keeping `$/` for
the two in-tree actions?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
