Skip to content

Use in-house shell completer classes. #156

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.

Sign up for GitHub

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

Jump to bottom
Closed
bckohan opened this issue Dec 4, 2024 · 0 comments
Closed

Use in-house shell completer classes. #156

bckohan opened this issue Dec 4, 2024 · 0 comments
Assignees
@bckohan
Labels
BREAKING This change will break downstream code. refactor A code refactor/redesign that does not noticeably effect functionality.

Comments

@bckohan
Copy link
Member

bckohan commented Dec 4, 2024 •
edited
Loading

Typer has repeatably broken shell completion logic and our use case is special enough (i.e. going through a management command so we do not need to rely on environment variables) that it makes more sense to implement the completer scripts and completion logic separately. We should still rely on typer to install the scripts for supported shells though.

This issue will help us bypass upstream problems causing #152
@bckohan bckohan added the refactor A code refactor/redesign that does not noticeably effect functionality. label Dec 4, 2024
@bckohan bckohan added this to the Version 3.0 milestone Dec 4, 2024
@bckohan bckohan self-assigned this Dec 4, 2024
@bckohan bckohan changed the title Take ownership of shell completion logic Use in-house shell completer classes. Dec 4, 2024
@bckohan bckohan added the BREAKING This change will break downstream code. label Dec 9, 2024
bckohan added a commit that referenced this issue Dec 9, 2024
@bckohan
start work on in house shell completer classes #156
1597b0f
bckohan added a commit that referenced this issue Dec 18, 2024
@bckohan
more work on #156
9312498
bckohan added a commit that referenced this issue Dec 18, 2024
@bckohan
zshell tests working again #156
775fe15
bckohan added a commit that referenced this issue Dec 19, 2024
@bckohan
add tests for terminal formatting in shell completion #156 #152
d6401e9
bckohan added a commit that referenced this issue Dec 31, 2024
@bckohan
fix bash completion scripts #156
94979b1
bckohan added a commit that referenced this issue Dec 31, 2024
@bckohan
force absolute paths on install of completion scripts #156
ed2296c
bckohan added a commit that referenced this issue Dec 31, 2024
@bckohan
run parser completer tests on current shell, or default to bash #156
5db4f95
bckohan added a commit that referenced this issue Dec 31, 2024
@bckohan
more shellcompletion CI work #156
f9b490a
bckohan added a commit that referenced this issue Dec 31, 2024
@bckohan
simply app stack for completion CI #156
c47a575
bckohan added a commit that referenced this issue Jan 2, 2025
@bckohan
first cut at getting powershell working #156
84510a1
bckohan added a commit that referenced this issue Jan 2, 2025
@bckohan
fix get_usage_script #156
ba86354
bckohan added a commit that referenced this issue Jan 3, 2025
@bckohan
powershell completion is working #156
14db8e1
bckohan added a commit that referenced this issue Jan 3, 2025
@bckohan
fix --settings= and --pythonpath= passthrough for bash and zsh, start…
53d46d7 
… work on powershell test #156
bckohan added a commit that referenced this issue Jan 4, 2025
@bckohan
powershell completion tests almost working #156
e5502f4
bckohan added a commit that referenced this issue Jan 5, 2025
@bckohan
update fish shell completion #156
4ce6e0f
bckohan added a commit that referenced this issue Jan 5, 2025
@bckohan
powershell completion tests working, more work on fish tests #156
2cb7a33
bckohan added a commit that referenced this issue Jan 5, 2025
@bckohan
fish completions working #156
0f7345f
bckohan added a commit that referenced this issue Jan 6, 2025
@bckohan
minor fixes #156
02bda8a
bckohan added a commit that referenced this issue Jan 7, 2025
@bckohan
add documentation for completer customization #156
e8c4558
bckohan added a commit that referenced this issue Jan 14, 2025
@bckohan
rearrange completion class imports, add tests for coverage #156
eff2e67
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bckohan bckohan

Labels
BREAKING This change will break downstream code. refactor A code refactor/redesign that does not noticeably effect functionality.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bckohan