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

Build scripts are not cross-platform #454

Closed
reduckted opened this issue Nov 6, 2017 · 1 comment · Fixed by #581
Closed

Build scripts are not cross-platform #454

reduckted opened this issue Nov 6, 2017 · 1 comment · Fixed by #581

Comments

@reduckted
Copy link
Contributor

The build scripts don't work on Windows. This makes it difficult for me (and others) to contribute to this project.

Some examples:

> yarn test
$ rimraf dist && tsc && cp -r test/fixtures dist/test && mocha dist/test --recursive
'cp' is not recognized as an internal or external command,
operable program or batch file.
error Command failed with exit code 1.
> yarn run prepare:package
$ cat package.json | ts-node build/package.ts > dist/package.json
'cat' is not recognized as an internal or external command,
operable program or batch file.
error Command failed with exit code 255.

It would be great if these scripts were cross-platform!

@mgechev
Copy link
Owner

mgechev commented Jan 7, 2018

@reduckted I'll fix this as part of the next release. Will add it to my schedule for next week.

@mgechev mgechev added this to the 4.1.0 - Manuel Blum milestone Jan 7, 2018
@mgechev mgechev modified the milestones: 4.1.0 - Manuel Blum, 4.1.1 - Anne Condon Jan 18, 2018
@mgechev mgechev modified the milestones: 4.1.1 - Anne Condon, 5.0.0 - Fred Brooks Feb 11, 2018
kevinbuhmann added a commit to kevinbuhmann/codelyzer that referenced this issue Apr 26, 2018
kevinbuhmann added a commit to kevinbuhmann/codelyzer that referenced this issue Apr 26, 2018
kevinbuhmann added a commit to kevinbuhmann/codelyzer that referenced this issue Apr 26, 2018
mgechev pushed a commit that referenced this issue Apr 26, 2018
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 a pull request may close this issue.

2 participants