ci: job conditions without || next to needs #9

Merged

noe.trevino merged 1 commit from fix-job-conditions into main

2026-10-03 22:45:30 -05:00

noe.trevino commented

2026-10-03 22:45:29 -05:00

Owner

In a called workflow, Forgejo rewrites needs.X and drops the parentheses in the job condition. The release job from #8 then ran on a pull request run in haunt.nvim. The docs job had the same fault.

The conditions now use only && and contains(fromJSON(...)). Tested on haunt.nvim#123: docs and release both skip on the PR run.

In a called workflow, Forgejo rewrites `needs.X` and drops the parentheses in the job condition. The release job from #8 then ran on a pull request run in haunt.nvim. The docs job had the same fault. The conditions now use only `&&` and `contains(fromJSON(...))`. Tested on haunt.nvim#123: `docs` and `release` both skip on the PR run.

noe.trevino added 1 commit

2026-10-03 22:45:29 -05:00

ci: job conditions without || next to needs b24efc8ddb

In a called workflow, Forgejo rewrites needs.X and drops the parentheses in
the job condition. The release job's condition became
'... || needs.docs.result == 'skipped'', so it ran on a pull request run.
The docs job had the same fault and was only safe because tests succeeded.

The conditions now use only && and contains(fromJSON(...)).

noe.trevino merged commit 449142410a into main

2026-10-03 22:45:30 -05:00

noe.trevino deleted branch fix-job-conditions

2026-10-03 22:45:30 -05:00

noe.trevino referenced this pull request from a commit

2026-10-03 22:45:30 -05:00

Merge pull request 'ci: job conditions without || next to needs' (#9) from fix-job-conditions into main

noe.trevino referenced this pull request from noe.trevino/haunt.nvim

2026-10-03 22:45:37 -05:00

ci: use the shared workflow with the release job #123