Skip to content

Fix #3098 #3274

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 5 commits into
base: main
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
Collaborator

@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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants