GitHub Community
-
|
I want to run a given workflow both on push and based on a cronjob. I assumed this to work: However that throws with: How can I get this to work? |
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 5 replies
-
|
When you want to specify properties for the events you need to specify as a map instead of an array. |
Beta Was this translation helpful? Give feedback.
-
|
When I try the accepted solution, the workflow editor seems to think the syntax is invalid [for having a string inside the yaml mapping vs a key with a trailing colon] (see screenshot). Is it possible that push here should instead be push:? I’ve tried using it with the colon in a quick test just now and both the push run and the schedule run executed as expected. |
Beta Was this translation helpful? Give feedback.
-
|
You can use
|
Beta Was this translation helpful? Give feedback.
-
|
Even though I’m a little late on this one, but yes is perfectly valid YAML. It’s all about the nesting. :slight_smile: |
Beta Was this translation helpful? Give feedback.
-
|
This is the only syntax that has worked for me |
Beta Was this translation helpful? Give feedback.
-
|
but something like on:
workflow_dispatch:
schedule:
- cron: '0 0 * * 1'Does not work, and neither does: on:
workflow_dispatch
schedule:
- cron: '0 0 * * 1'any suggestion? |
Beta Was this translation helpful? Give feedback.
-
|
That works fine https://github.com/chrispat/chrispat-action/blob/master/.github/workflows/use-action.yml. |
Beta Was this translation helpful? Give feedback.
-
|
I think you need to indent the |
Beta Was this translation helpful? Give feedback.
-
|
I think the extra-identation is not required. on:
schedule:
- cron: "0 0 * * 1"is the same as on:
schedule:
- cron: "0 0 * * 1"and should be the same as: on:
schedule: [{ "cron": "0 0 * * 1" }]YAML can be translated into JSON and vice-versa. (to some degree) The example above without the colon definitely does not work. I'm not sure why this on:
workflow_dispatch:
schedule:
- cron: '0 0 * * 1'does not work. |
Beta Was this translation helpful? Give feedback.
-
|
sorry I never replied that I got this working. My actual cron definition had an issue and github action just failed to understand it, but didn't explain why, and I thought it was related to the multiple targets syntax, but it was really the cron job not being understood. |
Beta Was this translation helpful? Give feedback.
When you want to specify properties for the events you need to specify as a map instead of an array.