fix(ci): correct invalid workflow configuration in master.yml #10805
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
b56abc4fac
|
fix(ci): correct invalid workflow configuration in master.yml
CI / push-validation (pull_request) Successful in 23s
CI / helm (pull_request) Successful in 34s
CI / build (pull_request) Successful in 3m48s
CI / lint (pull_request) Successful in 3m55s
CI / quality (pull_request) Successful in 4m20s
CI / typecheck (pull_request) Successful in 4m35s
CI / security (pull_request) Successful in 4m42s
CI / e2e_tests (pull_request) Successful in 7m7s
CI / unit_tests (pull_request) Successful in 7m16s
CI / integration_tests (pull_request) Successful in 7m47s
CI / docker (pull_request) Successful in 1m37s
CI / coverage (pull_request) Successful in 15m4s
CI / status-check (pull_request) Successful in 3s
CI / benchmark-regression (push) Waiting to run
CI / benchmark-publish (push) Waiting to run
CI / push-validation (push) Successful in 23s
CI / helm (push) Successful in 30s
CI / build (push) Successful in 3m51s
CI / lint (push) Successful in 3m57s
CI / quality (push) Successful in 4m21s
CI / security (push) Successful in 4m31s
CI / typecheck (push) Successful in 4m41s
CI / e2e_tests (push) Successful in 7m15s
CI / integration_tests (push) Successful in 7m52s
CI / unit_tests (push) Successful in 9m0s
CI / docker (push) Successful in 1m36s
CI / coverage (push) Successful in 14m58s
CI / status-check (push) Successful in 3s
CI / benchmark-regression (pull_request) Has been cancelled
CI / benchmark-publish (pull_request) Has been cancelled
Five issues were identified and fixed in .forgejo/workflows/master.yml:
1. Added 'pull_request' trigger to the 'on:' block so the
benchmark-regression job (which conditions on pull_request events)
can actually execute. Previously only 'push' was listed, making
the job permanently unreachable.
2. Removed the 'needs: [lint, typecheck, security, quality]' dependency
from benchmark-regression. Those jobs are defined in ci.yml, not in
master.yml, so referencing them here made the workflow structurally
invalid and would be rejected by Forgejo Actions at parse time.
3. Fixed malformed YAML in benchmark-publish: the ASV_S3_BUCKET
expression had a line break inside '${{ ... }}', which is invalid
YAML and caused a parse error.
4. Fixed 'rentention-days' typo to 'retention-days' in both the
benchmark-regression and benchmark-publish upload-artifact steps.
The misspelling caused the retention policy to be silently ignored.
5. Fixed step name typo 'Run asv ia nox' -> 'Run asv via nox' in the
benchmark-publish job.
ISSUES CLOSED: #10804
|