mgunyho / tere

Terminal file explorer
European Union Public License 1.2
1.67k stars 36 forks source link

Add shell instructions for nushell #94

Open Br1ght0ne opened 1 year ago

Br1ght0ne commented 1 year ago

https://nushell.sh

mgunyho commented 1 year ago

Thanks! This looks very similar to #89, looks like it doesn't handle options passed to tere, such as tere --version or tere --ignore-case, am I right?

Br1ght0ne commented 1 year ago

I did not notice #89 before opening this - sorry! Yes, this currently doesn't handle any options, as nushell complains about flags if they don't appear in the subcommand signature. I'll look into it, thanks!

ejenk0 commented 9 months ago

Hi this does not work for me, does anyone have a working config for using there in nushell? @Br1ght0ne

kubouch commented 8 months ago

We made some language changes to Nushell, and it should be possible to use def --wrapped --env to achieve both options passing and preserving the directory after cd.

mgunyho commented 8 months ago

Cool! Feel free to update the PR to include these. Maybe it's also good to mention that version > x.y.z is required, like it is saying for Xonsh currently.

I updated the README with some info on how the shell alias should work: https://github.com/mgunyho/tere#testing-a-new-shell-integration

Br1ght0ne commented 8 months ago

Thanks for the heads up @kubouch! I'll try it out in the coming days.