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

aws-sso.fish (line 2): Missing end to balance this function definition #472

Closed
dfroberg opened this issue Mar 10, 2023 · 0 comments · Fixed by #473
Closed

aws-sso.fish (line 2): Missing end to balance this function definition #472

dfroberg opened this issue Mar 10, 2023 · 0 comments · Fixed by #473
Labels
bug Something isn't working

Comments

@dfroberg
Copy link

Output of aws-sso version:

AWS SSO CLI Version 1.9.10 -- Copyright 2021-2022 Aaron Turner
11ec65288f78702a5e80078c04e0ef8b4ac7cb67 (v1.9.10) built at 2023-03-04T03:10:06+0000

Describe the bug:
Installation through the wizard of completion in fish errors out on every TAB.

To Reproduce:

  1. aws-sso
aws-sso console ~/.config/fish/completions/aws-sso.fish (line 2): Missing end to balance this function definition
function __complete_aws-sso
^~~~~~~^
from sourcing file ~/.config/fish/completions/aws-sso.fish
source: Error while reading file '/home/dfroberg/.config/fish/completions/aws-sso.fish'

Expected behavior:
Working completion

Desktop (please complete the following information):

  • OS: Ubuntu
  • Version 22.04

** Manual fix:**

  1. nano ~/.config/fish/completions/aws-sso.fish
  2. add end on blank line above # END_AWS_SSO_CLI
  3. Save and close

File should look like;

 cat ~/.config/fish/completions/aws-sso.fish
# BEGIN_AWS_SSO_CLI
function __complete_aws-sso
    set -lx COMP_LINE (commandline -cp)
    test -z (commandline -ct)
    and set COMP_LINE "$COMP_LINE "
    /home/linuxbrew/.linuxbrew/Cellar/aws-sso-cli/1.9.10/bin/aws-sso
end
# END_AWS_SSO_CLI
@dfroberg dfroberg added the bug Something isn't working label Mar 10, 2023
synfinatic added a commit that referenced this issue Mar 10, 2023
synfinatic added a commit that referenced this issue Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant