Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add fish syntax highlighting #820

Closed
aeosynth opened this issue May 19, 2016 · 12 comments
Closed

add fish syntax highlighting #820

aeosynth opened this issue May 19, 2016 · 12 comments
Labels

Comments

@aeosynth
Copy link

fish is a somewhat popular shell, and it would be nice if syntax highlighting were supported by default

is there a process for adding new languages to the default runtime?

@chrisbra
Copy link
Member

Someone, who already maintains a fish syntax script should state, the he wants to distribute his syntax script with vim. idealy the license allows us to distribute it (i think the vim license would be preferable). Do you know such a script (or perhaps even maintain such one)?

@kopischke
Copy link

Just in case nobody more qualified steps forward: I have recently forked the excellent, but unmaintained (the most up to date fork just integrates pending pull requests) vim-fish and started updating it (private branch, not pushed yet). I’m sure I am not the most qualified person to do so, but I would be willing to maintain it, or a syntax subset, for the Vim runtime if nobody else does.

@aeosynth
Copy link
Author

aeosynth commented Jun 9, 2016

@kopischke no one else seems interested, can you publish your code?

@OJFord
Copy link

OJFord commented Aug 31, 2017

It would be great if this could be added. I found this issue by trying to work through a mess of forks and determine which would be best to install. @wilriker and @fingerblaster have the most recently updated forks - I wonder if there's any interest there in maintaining an 'official' fork?

@wilriker
Copy link

wilriker commented Sep 3, 2017

On the one hand I'd like to provide my fork as an official source but to be honest, all I am doing here is trial&error most of the time as I am not exactly a master of vimscript. But I am willing to offer my code base to whoever feels more appropriate to maintain a vim plugin.

@rodrigobdz
Copy link

Just in case nobody more qualified steps forward: I have recently forked the excellent, but unmaintained (the most up to date fork just integrates pending pull requests) vim-fish and started updating it (private branch, not pushed yet). I’m sure I am not the most qualified person to do so, but I would be willing to maintain it, or a syntax subset, for the Vim runtime if nobody else does.

The link provided in the comment is broken. Would love to see Fish syntax highlighting in vim.

@MarcinOrlowski
Copy link

Seems @kopischke gave up as I am unable to find anything related vim-fish among his public repos.

@kopischke
Copy link

@MarcinOrlowski whoops, yes, sorry, I should have stated that but thought @wilriker would pick up the torch; my bad, this was not me at my best. I never pushed my half assed fork as RL has taken me away from regular Vim use, so consider me out of the equation. Again, apologies for not stating this clearly earlier.

@nickeb96
Copy link

nickeb96 commented Feb 2, 2023

I'd like to put my vim plugin for fish up as a candidate. The syntax highlighting is more accurate and consistent compared to other plugins I've seen. Also, it very closely matches how fish natively highlights interactive commands and output of the type builtin.

I will happily be a maintainer for these files and update them as fish changes and adds new features. I also haven't selected a license yet, so I can use the vim license if that helps.

The only question I have is how I go about submitting them. Do I just make a pull request with my additions? I couldn't find any relevant info on adding new runtime files in CONTRIBUTING.md.

@brammool
Copy link
Contributor

brammool commented Feb 2, 2023 via email

@nickeb96
Copy link

The fish runtime files above are now in vim as of this commit: 938ae28

It looks like someone with admin rights can close complete this issue.

@chrisbra
Copy link
Member

closing then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

10 participants