Status

[blankname]

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:

• Filter blank completions dag/vim-fish#48
• Define b:undo_ftplugin and fix b:match_words 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).
• Improve fish#Indent(). dag/vim-fish#29
  • Fantastic improvement! Thanks @Raimondi

Not merged:

• Fix indentation dag/vim-fish#46
  • Went with Fix indentation dag/vim-fish#46 as it was more comprehensive.
• Fix improper skipping of escaped backslashes before quotes dag/vim-fish#47
  • Made a different change that covers any number of backslashes + quotes correctly.
• Add an ftplugin for fish profile output files dag/vim-fish#24
  • See comments on the merge of 25.
• Clear autocmd to prevent pile-up of function declarations dag/vim-fish#43
  • This doesn't seem to be needed (the ftdetect autocommands don't seem to get doubled up).

Need to revisit:

• use standard shell definition for iskeyword 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]

Instead of merging dag#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.