rusackas commented on code in PR #44097:
URL: https://github.com/apache/superset/pull/44097#discussion_r3971868358
##########
.github/workflows/superset-helm-lint-test.yml:
##########
@@ -49,7 +49,7 @@
# allowlist (only v8.1.0+ are, at apache/infrastructure-actions'
# actions.yml). Needs an INFRA request before this can de-vendor too.
- name: Set up chart-testing
- uses: $/.github/actions/chart-testing-action
+ uses: ./.github/actions/chart-testing-action
Review Comment:
This one's a false positive here, not a fix. `$/` self-repository refs
aren't recognized by the ASF actions allowlist yet (upstream fix
apache/infrastructure-actions#1254 still open), so switching this back to `$/`
would reintroduce the exact CI break #44083 caused, which is literally what
this PR exists to fix. `./` is correct until that upstream fix lands.
--
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]
