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

workflow failing after changing to new syntax in entrypoint.sh #38

Closed
jazmc opened this issue Nov 30, 2022 · 3 comments
Closed

workflow failing after changing to new syntax in entrypoint.sh #38

jazmc opened this issue Nov 30, 2022 · 3 comments

Comments

@jazmc
Copy link

jazmc commented Nov 30, 2022

Hi!
First of all, thanks to all contributors for this extremely useful repository!

With the latest merge #37 where the syntax of entrypoint.sh was changed, our workflow stopped passing. Here are the last lines of GitHub actions workflow log:

(...)
- Uploading 0 files
✔ Finished uploading 0 assets
- Waiting for deploy to go live...
✔ Deploy is live!
Error: Unable to process file command 'env' successfully.
Error: Invalid format 'Deploying to main site URL...'

Then the workflow job fails, but from Netlify control panel you can see that the app has deployed successfully.

I guess this could be a syntax error in the echo statements, see discussion about a similar error here:
https://github.com/orgs/community/discussions/24952

@jsmrcaga
Copy link
Owner

Thanks Jasmin! I'm glad you find the repo useful and I'm so thankful to all the contributors as well.

It might be the multiline indeed. I feel a little dumb since I asked @h4k1 to use the single-line syntax 🙈 .
Would you be willing to try and see if the multi-line syntax fixes this @jazmc and open a PR? I'm not sure I'm gonna have the time in the coming days

@oshima-gangan
Copy link
Contributor

@jsmrcaga
Hi,
Thank you very much for letting me use convenient actions.

I have created a pull request with modified code, so please review it.

@jsmrcaga
Copy link
Owner

jsmrcaga commented Dec 9, 2022

Thanks @oshima-gangan , merged!

@jsmrcaga jsmrcaga closed this as completed Dec 9, 2022
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

No branches or pull requests

3 participants