[LTP] [COMMITTED][PATCH] ci/wiki-mirror: Fix triggering CI
Petr Vorel
pvorel@suse.cz
Mon Jan 17 11:13:43 CET 2022
Hi Xu,
> > FYI ba99b81708 ("doc: Fix typo missing s in .all_filesystems")
> > verified it's working:
> > https://github.com/linux-test-project/ltp/actions/runs/1707090366
> I found this problem when doc/* patch isn't the last patch of a
> patchset. But here it is a single patch.
Path check was removed in 820cfe539b ("ci/wiki-mirror: Don't check path").
Paths should be working not only on last commit, IMHO it evaluates all files in
push
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore
"When using the push and pull_request events, you can configure a workflow to
run based on what file paths are changed. Path filters are not evaluated for
pushes of tags."
=> it should be working even If I revert 820cfe539b, because running that
workflow does not take long.
Verification it's working on my local repo:
I pushed few commits in my local branch:
https://github.com/pevik/ltp/commits/tmp
the last commit was on .github/workflows/wiki-mirror.yml adding:
+ paths:
+ - 'doc/**'
commit only in that branch (not in upstream):
aa7ec06d22 ("Revert "ci/wiki-mirror: Don't check path"")
Other commit was modifying repo to 'pevik/ltp'
https://github.com/pevik/ltp/actions/runs/1699238259/workflow
...
on:
push:
branches:
- master
- tmp
paths:
- 'doc/**'
jobs:
mirror:
runs-on: ubuntu-latest
if: ${{ github.repository == 'pevik/ltp' }}
...
and that push triggered CI:
https://github.com/pevik/ltp/actions/runs/1699228985
(failed because my local repo does not have rights)
Kind regards,
Petr
More information about the ltp
mailing list