Make auth-syncserver build script work on Windows environments #107
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Solves
Running
pnpm build
results in a failure for the@localfirst/auth-syncserver
package on Windows environments. This is the error that's encountered:Description
A pre-cursor to this was to update the
pnpm-lock.yaml
file to the latest version, which is what I get when runningpnpm install
against themain
branch.I searched for suggestions on a cross-platform way to copy files in npm scripts and was guided towards shx by this question. This is a wrapper around ShellJS which is a collection of Unix commands implemented in node.js.
It's possible that we'd consider an alternative like copyfiles but I figured it was worth trying out something that could offer multiple commands for our future needs.
Testing
Verify that
pnpm build
still successfully copies the*.html
files to theauth-syncserver
build out.