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(server-ui):issue-67 Added Proper Escaping quotes in scripts:'build' to work in powershell #71

Conversation

bathini79
Copy link
Contributor

Issue: Dev server not starting
Resolution: its not starting from powershell cmd (Windows) Becaause of build in "build": "run-p "build:*"" .
Server is not able properly espace single quotes , this PR solves that .
Reference: https://stackoverflow.com/questions/70306554/pnpm-run-on-multiples-projects-based-on-location
image

@netlify
Copy link

netlify bot commented Aug 26, 2023

‼️ Deploy request for staging-firecamp-dev rejected.

Name Link
🔨 Latest commit c22251e

@bathini79 bathini79 changed the title fix: Added Proper Escaping quotes in scripts:'build' to work in power… fix: Added Proper Escaping quotes in scripts:'build' to work in pow Aug 26, 2023
@bathini79 bathini79 changed the title fix: Added Proper Escaping quotes in scripts:'build' to work in pow fix(server-ui):issue-67 Added Proper Escaping quotes in scripts:'build' to work in powershell Aug 26, 2023
@Nishchit14
Copy link
Contributor

If you'll search in the whole project "build": "run-p 'build:*'", you'll find other places too. would you mind to apply changes there too?

image

@bathini79
Copy link
Contributor Author

Sure, I can . Only these two were stopping to run the server. So, I fixed for these two services . I'll be doing up for other services in case

@bathini79
Copy link
Contributor Author

Done @Nishchit14

@Nishchit14 Nishchit14 merged commit b5eb24e into firecamp-dev:main Aug 31, 2023
@Nishchit14
Copy link
Contributor

#70 #67

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.

2 participants