vim-fish icon indicating copy to clipboard operation
vim-fish copied to clipboard

Status

Open blankname opened this issue 5 years ago • 1 comments

Since @dag is less active with vim-fish I decided it would be a good learning experience to try maintaining a fork and merging some prs.

Merged:

  • dag/vim-fish#48
  • dag/vim-fish#25
    • Merged the ftplugin change and not the added support for fish -p output files (although I could probably be convinced to merge that too, just need to think about it a bit more).
  • dag/vim-fish#29
    • Fantastic improvement! Thanks @Raimondi

Not merged:

  • dag/vim-fish#46
    • Went with dag/vim-fish#46 as it was more comprehensive.
  • dag/vim-fish#47
    • Made a different change that covers any number of backslashes + quotes correctly.
  • dag/vim-fish#24
    • See comments on the merge of 25.
  • dag/vim-fish#43
    • This doesn't seem to be needed (the ftdetect autocommands don't seem to get doubled up).

Need to revisit:

  • [x] dag/vim-fish#44
    • Originally I thought this was working without their change, but it turned out I had a plugin that was causing the desired behavior. I'll look into it again when I get the chance.

I've added a few other very minor miscellaneous changes as well (so feel free to browse the commit log).

Thanks @dag for creating vim-fish and all of the others for their contributions.

blankname avatar Jan 05 '20 21:01 blankname

Instead of merging dag/vim-fish#44, I've added a commit that just drops '/' from the iskeyword definition.

I didn't remove the iskeyword definition entirely because I believe including '_' and '.' is still useful.

I also defined syntax iskeyword to include '/' so that the syntax patterns using \k are unaffected.

blankname avatar Mar 22 '20 17:03 blankname