Add fish shell interpreter
This adds fish to the interpreters.
In a separate commit I also marked the fish entries as shell in case omitting it was an oversight. If that was a deliberate decision we can drop the commit.
Let me know if you have any concerns or need anything else from me to move this forward!
I'm not sure about either of the commits:
- fish isn't intended as a scripting language like bash is
-
shellis used by shellcheck for whichfish's syntax doesn't make sense
I'm not sure about either of the commits:
- fish isn't intended as a scripting language like bash is
I didn't see any kind of documentation on what did or did not belong in the interpreters, so I submitted this PR in hopes that pre-commit could run on my fish scripts. For what it's worth, fish documents itself as a scripting language and that is the main way I and other developers I know use it.
shellis used by shellcheck for whichfish's syntax doesn't make sense
Cool, that makes sense. If you think the other commit adding fish as an interpreter is beneficial feel free to just take that commit. If you would like me to handle it I can push a commit reverting the shell one, or push a new branch with just that one commit, or force-push to this branch to remove the shell commit -- whatever works for you. :)
And I forgot to say earlier that I found the identify codebase very approachable and easy to contribute to — thanks for making it that way!