Skip to content

feat(setqflist): include untracked files with 'all' target#1475

Merged
lewis6991 merged 1 commit intolewis6991:mainfrom
nemanjamalesija:feat/setqflist-include-untracked
Feb 4, 2026
Merged

feat(setqflist): include untracked files with 'all' target#1475
lewis6991 merged 1 commit intolewis6991:mainfrom
nemanjamalesija:feat/setqflist-include-untracked

Conversation

@nemanjamalesija
Copy link
Contributor

@nemanjamalesija nemanjamalesija commented Feb 4, 2026

Summary

  • When setqflist('all') is called and attach_to_untracked is enabled, untracked files are now included in the quickfix list
  • files_changed() accepts a new include_untracked parameter: uses ?? status from git status --porcelain (default base) or git ls-files --others --exclude-standard (custom base)
  • Updated docs for the "all" target to document the new behavior

Closes #829

Test plan

  • Set attach_to_untracked = true in config
  • Create an untracked file in the repo
  • Run :Gitsigns setqflist all and verify the untracked file appears in the quickfix list
  • Run with attach_to_untracked = false and verify untracked files are excluded
  • Test with a custom config.base to verify untracked files are included via ls-files

Assisted with claude code

@nemanjamalesija nemanjamalesija force-pushed the feat/setqflist-include-untracked branch from 126a4d5 to 3975999 Compare February 4, 2026 09:04
…enabled

When using setqflist('all'), untracked files are now included in the
quickfix list if the attach_to_untracked config option is enabled.

Closes lewis6991#829
@nemanjamalesija nemanjamalesija force-pushed the feat/setqflist-include-untracked branch from 3975999 to 2a6ffa0 Compare February 4, 2026 09:07
@lewis6991
Copy link
Owner

Why did you remove the "Assisted with claude code" from the PR description?

@nemanjamalesija
Copy link
Contributor Author

nemanjamalesija commented Feb 4, 2026

Why did you remove the "Assisted with claude code" from the PR description?

@lewis6991 I was not sure weather this would be something you would accept on the pull request

@lewis6991
Copy link
Owner

You should always include this disclosure.

@nemanjamalesija
Copy link
Contributor Author

You should always include this disclosure.

Ok, I updated the comment

@lewis6991 lewis6991 merged commit 1ce96a4 into lewis6991:main Feb 4, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Include untracked with setqflist

2 participants