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

Pass --unstable and --color always to fzf's default preview command #1758

Merged
merged 4 commits into from
Dec 11, 2023

Conversation

Qeole
Copy link
Contributor

@Qeole Qeole commented Dec 11, 2023

Pass --unstable to the preview command for the chooser, so that recipe previews can work even when the justfile relies on unstable features, such as !include directives.

Note that we're not taking the user by surprise by enabling unstable features in the preview: when the justfile includes unstable features, the user will need to pass --unstable explicitely to call just --choose --unstable already, or just will exit before launching the chooser.

Also tell just, in a second commit, to use colours for recipe previews.

Qeole and others added 4 commits December 11, 2023 22:46
Pass "--unstable" to the preview command for the chooser, so that
recipe previews can work even when the justfile relies on unstable
features, such as "!include" directives.

Note that we're not taking the user by surprise by enabling unstable
features in the preview: when the justfile includes unstable features,
the user will need to pass "--unstable" explicitely to call "just
--choose --unstable" already, or "just" will exit before launching the
chooser.
Tell just to use colours in the recipe preview for the default chooser.
@casey casey enabled auto-merge (squash) December 11, 2023 23:45
@casey
Copy link
Owner

casey commented Dec 11, 2023

Nice, good change! Merging now.

@casey casey merged commit e6e1823 into casey:master Dec 11, 2023
5 checks passed
@Qeole
Copy link
Contributor Author

Qeole commented Dec 11, 2023

Thanks!

@Qeole Qeole deleted the pr/chooser-opts branch December 11, 2023 23:46
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