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

feat: support shell("fish") #663

Open
Tracked by #1944
kemingy opened this issue Jul 26, 2022 · 15 comments · May be fixed by #1952
Open
Tracked by #1944

feat: support shell("fish") #663

kemingy opened this issue Jul 26, 2022 · 15 comments · May be fixed by #1952

Comments

@kemingy
Copy link
Member

kemingy commented Jul 26, 2022

Description

fish is another modern shell that works out of the box. It would be good to support the following command in envd.

def build():
    base(os="ubuntu20.04")
    shell("fish")

Message from the maintainers:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@kemingy
Copy link
Member Author

kemingy commented Sep 11, 2022

Requirements:

  • apt install fish
  • starship init fish
  • fish config (conda init if it's required)

@kemingy
Copy link
Member Author

kemingy commented Sep 16, 2022

Also, it will be good if can support fish shell autocomplete.

@gaocegege gaocegege removed the good first issue ❤️ Good for newcomers label Oct 7, 2022
@Belyenochi
Copy link
Contributor

If no one else, please assign it to me :-).

@kemingy
Copy link
Member Author

kemingy commented Oct 9, 2022

/assign @Belyenochi

@gaocegege
Copy link
Member

It may be not easy. Feel free to ask here if you encountered any problem!

Thanks for your contribution! 🎉 👍

@Belyenochi
Copy link
Contributor

Belyenochi commented Oct 10, 2022

@gaocegege @kemingy I'm trying to re-understand the problem, from what I've tried, if I run something like the following,

// build.envd
def build():
    base(os="ubuntu20.04", language="python3")
    shell("fish")
    config.jupyter()

envd will install fish in the build ubuntu image, and I understand what we need to do is modify /pkg/lang/ir /custom.go or similar file to support installed in the default image and initializes fish related configuration (including autocompletion plugins?)

@kemingy
Copy link
Member Author

kemingy commented Oct 10, 2022

envd will install fish in the build ubuntu image, and I understand what we need to do is modify /pkg/lang/ir /custom.go or similar file to support installed in the default image and initializes fish related configuration (including autocompletion plugins?)

  • Yes, we will use shell("fish")
  • Need to modify several files, I think you can search "zsh" in the codebase as a reference (no need to install oh-my-fish or something similar)
  • envd fish autocomplete should be placed in pkg/autocomplete/fish.go

@Belyenochi
Copy link
Contributor

envd will install fish in the build ubuntu image, and I understand what we need to do is modify /pkg/lang/ir /custom.go or similar file to support installed in the default image and initializes fish related configuration (including autocompletion plugins?)

  • Yes, we will use shell("fish")
  • Need to modify several files, I think you can search "zsh" in the codebase as a reference (no need to install oh-my-fish or something similar)
  • envd fish autocomplete should be placed in pkg/autocomplete/fish.go

Thanks for your patience :-).

@kemingy
Copy link
Member Author

kemingy commented Nov 10, 2022

Hi @Belyenochi, it has been a while. May I know if you're still working on this issue?

Feel free to ask for help.

@Belyenochi
Copy link
Contributor

Belyenochi commented Nov 11, 2022

Hi @Belyenochi, it has been a while. May I know if you're still working on this issue?

Feel free to ask for help.

Hi @kemingy , I'm working on it, maybe a PR in two weeks

@kemingy
Copy link
Member Author

kemingy commented Nov 11, 2022

Hi @Belyenochi, it has been a while. May I know if you're still working on this issue?
Feel free to ask for help.

Hi @kemingy , I'm working on it, maybe a PR in two weeks

Thanks

@kemingy
Copy link
Member Author

kemingy commented May 29, 2023

Hi @Belyenochi any update?

@Belyenochi
Copy link
Contributor

@kemingy Sorry for not updating yet, if the priority is not high, I will continue to take time to complete it.

@kemingy
Copy link
Member Author

kemingy commented Sep 6, 2023

Hi @Belyenochi any progress?

@kemingy kemingy mentioned this issue Jan 14, 2025
13 tasks
@kemingy
Copy link
Member Author

kemingy commented Jan 14, 2025

It's a bit easier now since fish 4 can be delivered as a single binary file: https://github.com/fish-shell/fish-shell/releases/tag/4.0b1

@kemingy kemingy linked a pull request Jan 18, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging a pull request may close this issue.

3 participants