Skip to content
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

bug: Invalid workflow file #239

Closed
pythoninthegrass opened this issue Dec 21, 2023 · 0 comments · Fixed by #240
Closed

bug: Invalid workflow file #239

pythoninthegrass opened this issue Dec 21, 2023 · 0 comments · Fixed by #240

Comments

@pythoninthegrass
Copy link
Contributor

In the outputs example usage, this is the most likely deprecated GitHub Actions syntax:

- name: feedback
        if: steps.automerge.outputs.mergeResult == "merged"
        run: |
          echo "Pull request ${{ steps.automerge.outputs.pullRequestNumber }} merged!"

Whereas working conditional code uses yaml curly brackets:

- name: feedback
        if: ${{ steps.automerge.outputs.mergeResult == 'merged' }}
        run: |
          echo "Pull request ${{ steps.automerge.outputs.pullRequestNumber }} merged!"

Full error

Invalid workflow file: .github/workflows/automerge.yml#L48
The workflow is not valid. .github/workflows/automerge.yml (Line: 48, Col: 13): Unexpected symbol: '"merged"'. Located at position 40 within expression: steps.automerge.outputs.mergeResult == "merged"

pythoninthegrass added a commit to pythoninthegrass/automerge-action that referenced this issue Dec 21, 2023
Fixes pascalgn#239 by using GitHub flavored yaml syntax for an in-line conditional.
pascalgn pushed a commit that referenced this issue Jan 8, 2024
Fixes #239 by using GitHub flavored yaml syntax for an in-line conditional.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant