Skip to content

modules/zsh: more settings to costumize completions#136

Merged
zimward merged 1 commit intoLassulus:mainfrom
Mrid22:better-completions
Feb 26, 2026
Merged

modules/zsh: more settings to costumize completions#136
zimward merged 1 commit intoLassulus:mainfrom
Mrid22:better-completions

Conversation

@Mrid22
Copy link

@Mrid22 Mrid22 commented Feb 25, 2026

New stuff:
option for case insensitive completion
option to add zsh-completions package for extra completions option to have color for file completions
option to add color to directory completions
option to integrate fzf-tab for fuzzy search completion

@Mrid22 Mrid22 force-pushed the better-completions branch from e15044a to af5f6e9 Compare February 25, 2026 11:36
@Mrid22 Mrid22 changed the title Added more compleion options modules/zsh: more settings to costumize completions Feb 25, 2026
Copy link
Collaborator

@zimward zimward left a comment

Choose a reason for hiding this comment

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

generally looks fine, but the commit should start with module/zsh like in the other pr already mentioned. the idea is generally to follow nixpkgs' commit convention

New stuff:
option for case insensitive completion
option to add zsh-completions package for extra completions
option to have color for file completions

fzf tab support
@Mrid22 Mrid22 force-pushed the better-completions branch from af5f6e9 to 098bd27 Compare February 26, 2026 05:57
@Mrid22
Copy link
Author

Mrid22 commented Feb 26, 2026

So im probably reading the test logs wrong somehow, but it looks like this failed because they all run as one action and the mpv check failed even though that module wasn't even updated in this pr?

@zimward
Copy link
Collaborator

zimward commented Feb 26, 2026

welcome to github where restarting a CI job three times in a row makes it go from red to green

@zimward zimward merged commit c954a7c into Lassulus:main Feb 26, 2026
3 of 4 checks passed
@Mrid22 Mrid22 deleted the better-completions branch March 1, 2026 17:03
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