-
Notifications
You must be signed in to change notification settings - Fork 363
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: some tasks fail on Windows due to missing shell commands #781
Conversation
@eladb I also tried to "refactor" by extracting the code, but it becomes gnarly so I'd rather let it be the way it is. |
Codecov Report
@@ Coverage Diff @@
## main #781 +/- ##
==========================================
+ Coverage 78.46% 82.67% +4.20%
==========================================
Files 88 96 +8
Lines 3181 3544 +363
Branches 809 793 -16
==========================================
+ Hits 2496 2930 +434
+ Misses 683 612 -71
Partials 2 2
Continue to review full report at Codecov.
|
Oh man, this extremely frustrating. We ditched projen because we are on a mixed team with mostly Windows developers. Overall kind of sad, because projen would help so much, especially inexperienced TypeScript/Javascript folks on Windows. |
This PR tries to address #452 by wrapping the commands 'rm', 'mkdir' and 'mv' with
shx
.A whitelist of commands is maintained in the task runtime.
@eladb Is this what you had in mind when commenting on #540?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.