-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
Define new setting: "git.statusArgs.untrackedFiles" #136698
Define new setting: "git.statusArgs.untrackedFiles" #136698
Conversation
This could also be an alternate approach to fixing #131020. I think landing both changes would be ideal. I am more than happy to rebase this change for compatibility, if there's interest. |
9685bc7
to
dbdeec1
Compare
I have rebased this for compatibility with the recently landed change, #131030. |
@ajhyndman, thank you very much for rebasing this pull request. I have looked at both pull requests in this area and I have decided to merge #131030 as that adds support for |
@lszomoru I work with an oversized monorepo in my day job. I frequently patch the vscode git extension to set the status arg to This is definitely still not an ideal workflow, but |
@ajhyndman, thank you very much for getting back to me on this. I think that your scenario is valid, my only concern is that we do not want to lift |
@lszomoru That makes perfect sense. I agree that this is an awkwardly low-level setting! Do you think it would make any more sense to expose the option programmatically instead? I'd be happy to write myself a small plugin to get this functionality rather than have to manually patch the built-in git extension. |
Closing this pull request in favour of #169253 |
This PR fixes #128800
If anyone has suggestions for how I might opt into this behaviour without patching the built-in git extension, I'd love to hear that as well. Without this change, VS Code's git features are almost unusable in my team's large repository.
NOTE: With this change, when using
--untracked-files=normal
, the VS Code status panel will not render untracked folders. It's an interesting product question whether or how untracked folders might be communicated in this mode, but simply omitting untracked folders vastly improves my workflow now. I don't see any reason that this need be a blocking decision.