search

jarun / nnn

n³ The unorthodox terminal file manager
BSD 2-Clause "Simplified" License
19.1k stars 760 forks source link

quitcd: fix range limits for new nushell version #1890

Closed BeyondMagic closed 4 months ago

BeyondMagic commented 4 months ago

Last version of nushell changed the way ranges work, so the command str substring 4..-1 isn't behaving the way it was anymore.

N-R-K commented 4 months ago

Won't this break for people who have older nushell versions or is the new patch backwards compatible? If it's not backwards compatible, then please document the minimum required version in the comment at the top of the file.

BeyondMagic commented 4 months ago

Won't this break for people who have older nushell versions or is the new patch backwards compatible?

No; not in the way it is coded right now.

BeyondMagic commented 4 months ago

Resolved with backwards compatibility and fixed trimming issue.

jarun commented 4 months ago

Thank you!