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 #3098 #3274

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fix #3098 #3274

wants to merge 5 commits into from

Conversation

fr-esco
Copy link

@fr-esco fr-esco commented Oct 7, 2022

Checklist

  • Issue number for this PR: spawn gcloud ENOENT #3098
  • Docs included?: No
  • Test units included?: No
  • In a clean directory, yarn install, yarn test run successfully? Yes

Description

Running the deploy schematic on Windows:

  1. Child processes that need gcloud cannot properly locate it;
  2. The path to main.js is printed escaped by \\ instead of /, even though target runtime is Linux based.

Code sample

N. A.

Copy link
Member

@davideast davideast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fr-esco Thank you so much and our apologies for taking so long to get to it. Since this is an older PR and there are a lot of merging PRs in flight it's going to take a bit to get this tested and merged so hang tight.

@davideast davideast self-assigned this Jun 23, 2023
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 this pull request may close these issues.

None yet

2 participants