Questions about the build workflow, github actions not triggered

108 views
Skip to first unread message

Florian Giroud

unread,
Feb 2, 2021, 3:37:05 AM2/2/21
to OpenRefine Development
Antonin, Kush, I am curious about our CI

My recent pushes to master have not triggered any Cypress run for the "snapshot_release"
That's problematic

But few days ago, some pushes have triggered a snapshot release
See in the github actions panel

Did something changed ?
Or is it because I used "'squash and merge" ?

Thanks in advance
Best,
- Florian

Kush Trivedi

unread,
Feb 2, 2021, 3:43:44 AM2/2/21
to openref...@googlegroups.com
Hi Florian,
Thanks for noticing, looking into it looks like a recent PR for paths-ignore has broken the workflow. Taking from the github-actions doc, when we only need to exclude the paths we should use `paths-ignore` rather than negation sign, and if we are using positive and negative patterns then we should define a positive matching pattern too https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#excluding-paths.

I will make the fix for it quickly.

Regards,
Kush

--
You received this message because you are subscribed to the Google Groups "OpenRefine Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openrefine-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openrefine-dev/9e63f87e-0a28-49d7-9d62-0168156661c3n%40googlegroups.com.

Thad Guidry

unread,
Feb 2, 2021, 9:59:16 AM2/2/21
to openref...@googlegroups.com
But we don't want snapshots being generated when only the files under /docs have changed.
Snapshot releases should only be generated on source code changes I would think (maybe we should add those positive also)?
So are not the "paths" with negative and positive patterns the right approach?



Kush Trivedi

unread,
Feb 2, 2021, 10:04:31 AM2/2/21
to openref...@googlegroups.com
Hi Thad,

For ignoring a directory, github has 'paths-ignore' but however if you want to choose negative and positive, then you also need to explicitly specify what paths you need as positive and negative. In this case, when we only want to ignore 'docs' directory we should have gone ahead with paths-ignore. Thats what I was able to get from what github-actions docs specified.

Also, Github workflow for snapshot is restored now.

Regards,
Kush

Reply all
Reply to author
Forward
0 new messages