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 windows build #9694

Closed
wants to merge 4 commits into from
Closed

Conversation

jondlm
Copy link
Contributor

@jondlm jondlm commented May 4, 2024

↪️ Pull Request

Fixes the windows build by using cross platform shx alternatives to plain commands like cp that were failing CI.

💻 Examples

From my windows box:
image

🚨 Test instructions

yarn build on a windows machine

@yamadapc yamadapc self-requested a review May 6, 2024 03:31
@yamadapc
Copy link
Contributor

yamadapc commented May 6, 2024

Could you use an unix shell instead of having these changes?

I suppose this worked on CI previously because it has cp and rm on windows (unix like bash shell)?

@jondlm
Copy link
Contributor Author

jondlm commented May 6, 2024

Huh yeah, I thought I was seeing failures on CI related to this but I think I misread them. I was only seeing them locally. Yeah I can use an alternate shell that has them available. I'll close this.

@jondlm jondlm closed this May 6, 2024
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