-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(gh action): use correct workflow yaml syntax #5523
Conversation
Even though this is a stub, it should have the correct syntax with the dash indicating the mapping for "uses"
👷 Deploy request for cypress-docs pending review.Visit the deploys page to approve it
|
|
✅ Deploy Preview for benevolent-cat-040f48 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
According to https://www.yamllint.com/ it is valid YAML: That is only because there is only one jobs:
cypress-run:
steps:
- uses: actions/checkout@v4
- uses: cypress-io/github-action@v6 whereas this is not # invalid example: don't copy!
jobs:
cypress-run:
steps:
- uses: actions/checkout@v4
uses: cypress-io/github-action@v6
# leads to "Map keys must be unique" error So, even though the original example is valid YAML, I support making your suggested change, which would make the example more general. |
Huh, TIL. Thanks for the feedback! |
Actually TIL for me too, but only after I researched the topic. 🙂 |
Hi @schlich 👋, thanks for the contribution! |
Even though this is a stub, it should have the correct syntax with the dash indicating the mapping for "uses"