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 "--diff
implies --no-stash
" feature
#1303
Conversation
--diff
implies --no-stash
" feature
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.
Thanks for the PR! It does need some changes, however.
d79eb83
to
004945e
Compare
4641c36
to
6320e92
Compare
This reverts commit 004945e.
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #1303 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 26 26
Lines 740 740
Branches 198 198
=========================================
Hits 740 740 ☔ View full report in Codecov by Sentry. |
closes #1279
Related work: #1250, #1259.
Current result
When only
--diff
is passed,--stash
is stilltrue
. It is set at line 65, stored in options at line 105 and passed to index.js at line 127.Expected result
When only
--diff
is passed,--stash
should befalse
.