-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Forcing the creation of tmp directory #1856
Conversation
Please sign your commits following these rules: $ git clone -b "master" [email protected]:arthrp/cli.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f Amending updates the existing PR. You DO NOT need to open a new one. |
Codecov Report
@@ Coverage Diff @@
## master #1856 +/- ##
==========================================
- Coverage 56.75% 56.75% -0.01%
==========================================
Files 309 309
Lines 21672 21658 -14
==========================================
- Hits 12301 12292 -9
+ Misses 8472 8469 -3
+ Partials 899 897 -2 |
@arthrp could you sign-off and amend your commit? |
… it already exists Signed-off-by: Arthur Peka <[email protected]>
@thaJeztah no problem, hope it's good now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
SGTM (i'm not john hodgman) |
Anything more to be done from my side? |
Looks like this one dropped off the radar; still LGTM |
Currently make.ps1 fails when 'tmp' directory already exists (e.g. due to the failed build).
You get error such as:
Passing -Force flag fixes that.