mirror of
https://github.com/nektos/act.git
synced 2026-04-25 17:05:50 +03:00
[GH-ISSUE #545] Issue: schedule configuration not validated, allowing invalid syntax #369
Labels
No labels
area/action
area/cli
area/docs
area/image
area/runner
area/workflow
backlog
confirmed/not-planned
kind/bug
kind/discussion
kind/external
kind/feature-request
kind/question
meta/duplicate
meta/invalid
meta/need-more-info
meta/resolved
meta/wontfix
meta/workaround
needs-work
pull-request
review/not-planned
size/M
size/XL
size/XXL
stale
stale-exempt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/act#369
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Stretch96 on GitHub (Feb 24, 2021).
Original GitHub issue: https://github.com/nektos/act/issues/545
Hello 👋
Found what may be either a bug, or might be an intentional
If
actis to be used as a way to verify workflow syntax, then I think this might be a bugactwill succeed with invalidschedulesyntaxAct version
Expected behaviour
actshould fail because GitHub expectsscheduleto be a listRunning this on GitHub, the error is shown:
Actual behaviour
actignoresschedulereferences by default as far as I can tellWorking workflow
Difference being the
-, which makesschedulea list rather than map. Which is what GitHub expects@github-actions[bot] commented on GitHub (Mar 27, 2021):
Issue is stale and will be closed in 14 days unless there is new activity
@ZauberNerd commented on GitHub (Feb 1, 2022):
Check out https://github.com/rhysd/actionlint to lint your workflow files.
@catthehacker commented on GitHub (Mar 25, 2022):
@nektos/act-maintainers do we want to implement that?
@KnisterPeter commented on GitHub (Mar 25, 2022):
Not sure, we are up to now, don't validate the workflow files. We talked about integrating actionlint for that.
I guess validating cron should belong in there.