-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
Staging when ignoring whitespace, doesn't stage changed whitespace #63397
Comments
(Experimental duplicate detection) |
|
Yeah, if I show whitespace changes, then it works. But I'm asking, why does it work that way? Meaning, if I have whitespace changes hidden, and I stage a range, then the textual changes are staged, but the whitespace changes are not staged. So I end up with a commit that has changes committed without any indentation, so I have to then stage and commit the indentation separately. Having whitespace changes hidden definitely makes it easier to identify changes, but having to remember to turn it on before any staging every time, is a bit tedious. |
To anyone arriving from Google, the current workaround is |
@joaomoreno whitespace changes should be displayed (i.e. not ignored) by default. For Python , whitespace changes are a critical syntatic and semantic aspect of the code . |
Thanks @teresaejunior - what a pain in the ass, will use work around. I noticed using selective diff was breaking and had to revert or append commits often. This is currently pretty bad UX :/ |
…Whitespace": false to make indentation changes appear at all when committing, stupid, it should fix the build: microsoft/vscode#63397 (comment)
Issue Type: Bug
if (true)
staged, since it's a text change, but the block contained within its braces as not staged, since it's just an indentation change.VS Code version: Code 1.29.1 (bc24f98, 2018-11-15T19:06:21.742Z)
OS version: Darwin x64 18.0.0
The text was updated successfully, but these errors were encountered: