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

The git_files picker falls back to find_files when outside of a git repository #1342

Closed
wants to merge 7 commits into from
Closed

The git_files picker falls back to find_files when outside of a git repository #1342

wants to merge 7 commits into from

Conversation

marekdedic
Copy link

Optionally can be disabled with the find_files_fallback option. Continuation of the discussion in #1338.

This makes https://github.com/nvim-telescope/telescope.nvim/wiki/Configuration-Recipes#falling-back-to-find_files-if-git_files-cant-find-a-git-directory obsolete

Comment on lines +149 to +150
---@field find_files_fallback boolean: if true, the find_files picker is run instead when the picker is used
-- outside of a git repository (default: true)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the docs be split this way?

@Conni2461
Copy link
Member

Conni2461 commented Oct 14, 2021

I think we shouldnt do this.

No other builtin references another builtin. Doing this here makes it less self contained. The solution presented is less elegant than what users can do in a simple function.

@marekdedic
Copy link
Author

Hi,
check #1338 as for my reason for wanting it. Honestly, I can't imagine why you'd ever want to use the git_files picker without this feature. It's up to you to decide if it's worth the additional complexity or if you want to push this complexity to the users...

@Conni2461
Copy link
Member

I still dont like it. I think it doesnt fit with potential plans of a modular telescope: #1228

@marekdedic
Copy link
Author

Ok, closing.

@marekdedic marekdedic closed this Nov 16, 2021
@marekdedic marekdedic deleted the git-files-find-files-fallback branch November 16, 2021 14:00
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.

2 participants