mgunyho / tere

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

correct a possible error #33

Closed akkartik closed 2 years ago

akkartik commented 2 years ago

To my knowledge aliases and functions don't create subshells.

mgunyho commented 2 years ago

Hmm, you're right. I guess the thinking behind this wording was that it wouldn't work with a simple alias, but looks like at least alias t='cd $(tere)' works in bash. Maybe we can leave the "script" out, what I want to highlight here is that the shell config is necessary. So I would change it to

It has to be usually done using a function or alias, since a subprocess cannot change the working directory of the parent.

akkartik commented 2 years ago

Makes sense. I'll make the edit.

mgunyho commented 2 years ago

Thanks!