adoroszlai commented on code in PR #544:
URL: https://github.com/apache/ozone-site/pull/544#discussion_r3870804603
##########
.github/workflows/asf-allowlist-check.yaml:
##########
@@ -45,6 +45,7 @@ jobs:
with:
persist-credentials: false
- name: Check actions
- uses:
apache/infrastructure-actions/allowlist-check@69afc125e535c4c41e7f1b7470f583087e0f344b
+ # yamllint disable-line rule:line-length
Review Comment:
Yes, it's needed to prevent yamllint
[failure](https://github.com/adoroszlai/ozone-site/actions/runs/33047440255/job/98434655500#step:6:13):
```
./.github/workflows/asf-allowlist-check.yaml
48:121 error line too long (123 > 120 characters) (line-length)
```
I tried to [break the long
line](https://github.com/adoroszlai/ozone-site/commit/1a20893f27be84a393bfbc458b99d9c3d3ccf926),
but the workflow
[failed](https://github.com/adoroszlai/ozone-site/actions/runs/33048042930/job/98436585820#step:1:24)
elsewhere:
```
Error: Unable to resolve action
`apache/infrastructure-actions@61dcea11f19e2bbe1263f14d72235e8da17d3ad0 #
allowlist-check/v1.0.0`, unable to find version
`61dcea11f19e2bbe1263f14d72235e8da17d3ad0 # allowlist-check/v1.0.0`
```
I took it from similar change in `airflow`:
https://github.com/apache/airflow/blob/9ad206a01def2dc6fffe9a7d237c643d66645078/.github/workflows/asf-allowlist-check.yml#L34-L35
--
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]
