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

fix(action): set entrypoint type to string #27

Merged
merged 1 commit into from
Jul 17, 2020

Conversation

ShahradR
Copy link
Owner

Update the action.yaml file to pass the entrypoint parameter as a single string, rather than a sequence of arguments.

act expects the entrypoint to be of type string[]. However, the GitHub runner expects type string. Because of this disconnect, workflow files that pass their act tests throw exceptions when ran on GitHub hosted runners, and vice-versa.

To work around this issue, the act JUnit tests have been temporarily ignored until this issue can be resolved.

To see the entrypoint parser in act, see https://git.io/JJZx0
To see the entrypoint parter in the GH runner, see https://git.io/JJZxV

Associated issue: #26

Update the action.yaml file to pass the `entrypoint` parameter as a
single string, rather than a sequence of arguments.

act expects the entrypoint to be of type string[]. However, the GitHub
runner expects type string. Because of this disconnect, workflow files
that pass their act tests throw exceptions when ran on GitHub hosted
runners, and vice-versa.

To work around this issue, the act JUnit tests have been temporarily
ignored until this issue can be resolved.

To see the entrypoint parser in act, see https://git.io/JJZx0
To see the entrypoint parter in the GH runner, see https://git.io/JJZxV

Associated issue: #26
@ShahradR ShahradR added the bug Something isn't working label Jul 17, 2020
@ShahradR ShahradR self-assigned this Jul 17, 2020
@ShahradR ShahradR merged commit 328e512 into develop Jul 17, 2020
@ShahradR ShahradR deleted the bugfix/entrypoint-sequence branch July 17, 2020 22:59
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.0.1-develop.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

ShahradR added a commit that referenced this pull request Nov 29, 2020
feat: add Dependabot support to the project
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant