gazorby / fish-abbreviation-tips

💡 Help you remembering your abbreviations
MIT License
299 stars 7 forks source link

Add '--' to all calls to `string` #16

Closed matiasjrossi closed 3 years ago

matiasjrossi commented 3 years ago

Description

Initially looked into this to solve the following bug:

❯ ls \
      --foobarstring trim: Unknown option '--foobar'

~/.config/fish/functions/__abbr_tips_bind_newline.fish (line 1):
in command substitution
    called on line 3 of file ~/.config/fish/functions/__abbr_tips_bind_newline.fish
in function '__abbr_tips_bind_newline'

(Type 'help string' for related documentation)

ls: unrecognized option '--foobar'
Try 'ls --help' for more information.

Then went ahead and made the same changes across the plugin.

With the changes to functions/__abbr_tips_bind_newline.fish the error message is prevented:

❯ fisher install matiasjrossi/fish-abbreviation-tips
fisher install version 4.3.0
Fetching https://codeload.github.com/matiasjrossi/fish-abbreviation-tips/tar.gz/HEAD
Installing matiasjrossi/fish-abbreviation-tips
           /home/matiasr/.config/fish/functions/__abbr_tips_bind_newline.fish
           /home/matiasr/.config/fish/functions/__abbr_tips_bind_space.fish
           /home/matiasr/.config/fish/functions/__abbr_tips_init.fish
           /home/matiasr/.config/fish/conf.d/abbr_tips.fish
Installed 1 plugin/s

❯ ls \
      --foobar
ls: unrecognized option '--foobar'
Try 'ls --help' for more information.

❯ git status
fatal: not a git repository (or any of the parent directories): .git

💡 gs => git status

Related issues

N/A

Notes

Tests passing.

gazorby commented 3 years ago

Thanks @matiasjrossi!