Skip to content
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

Debug target selection interface should not respect .gitignore #6481

Closed
BCMM opened this issue Mar 29, 2023 · 0 comments · Fixed by #7936
Closed

Debug target selection interface should not respect .gitignore #6481

BCMM opened this issue Mar 29, 2023 · 0 comments · Fixed by #7936
Labels
C-bug Category: This is a bug

Comments

@BCMM
Copy link

BCMM commented Mar 29, 2023

Summary

Helix respects .gitignore for the purposes of tab-completion when selecting a debug target. While this makes sense for probably every other file selection in Helix, in this case, it makes it harder to quickly select the correct file.

While I've given reproduction steps for Cargo projects, this problem is likely to occur with many languages and build systems, since it is normal to .gitignore build directories.

Reproduction Steps

I tried this:

  1. Ensure debugging is available (as per hx --health rust)
  2. Build a typical Cargo project (such as Helix itself, or the Hello World generated by cargo new), then open any source file in Helix
  3. <space> g l
  4. Select "binary" or "binary (terminal)"
  5. Try to enter the path of the executable using tab completion

I expected this to happen:

Helix should allow tab-completion of the target/ directory.

Instead, this happened:

Any other file or directory can be tab-completed, but target/ must be typed manually, because it is in .gitignore.

Helix log

No response

Platform

Linux

Terminal Emulator

Konsole

Helix Version

22.12-497-g5b3dd6a6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant