Skip to content

Feat/bash completion - #317

Merged
joto merged 4 commits into
osmcode:masterfrom
tuesd4y:feat/bash-completion
Aug 3, 2026
Merged

Feat/bash completion#317
joto merged 4 commits into
osmcode:masterfrom
tuesd4y:feat/bash-completion

Conversation

@tuesd4y

@tuesd4y tuesd4y commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

I've added a quick implementation for the bash completion (#297).

I've also added the new query-locations-index and create-locations-index to the zsh completions, since they weren't in there before.

Didn't take a stab at the fish completions, since I've never used that shell.

@joto

joto commented Aug 1, 2026

Copy link
Copy Markdown
Member

This looks pretty good, found just one problem: opl|opl.gz|opl.bz2 should also be allowed as input formats. That's wrong in the zsh format already. (Guess I never added that after OPL was implemented as input format).

Can you add some information in the file header on where/how to install the file?

@tuesd4y

tuesd4y commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

This looks pretty good, found just one problem: opl|opl.gz|opl.bz2 should also be allowed as input formats. That's wrong in the zsh format already. (Guess I never added that after OPL was implemented as input format).

Added

Can you add some information in the file header on where/how to install the file?

I've just included both completions it in the make install script, does that work for you?

@joto

joto commented Aug 3, 2026

Copy link
Copy Markdown
Member

Thank you!

@joto
joto merged commit e43e7c9 into osmcode:master Aug 3, 2026
26 checks passed
@tuesd4y
tuesd4y deleted the feat/bash-completion branch August 3, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants