Tim Pope

First question: Why `autoload/_fireplace.lua`/`require'_fireplace'`? Is this a convention?

That answers half the question. Why `autoload/`? It's not autoloaded. And Neovim seems to prefer `lua/`.

My bad, could have sworn I saw `autoload/_fireplace.lua` in the diff. If the function changes, how do I reload it? I think ideally, reloading `autoload/fireplace.vim` would also reload the lua...

You're not alone; this is a common pain point. I'm updating the README to point directly to the usage documentation. > For example, if I highlight a function name and...

> When there's a hyphen, `[d` reads only up until it. That's `'iskeyword'`, which _should_ be set correctly by the ftplugin. Make sure `:filetype` reports `plugin:ON` and check `:verbose set...

I consider cider-nrepl mandatory for cljs. I think the appropriate fix is a better error message.

Can anyone still reproduce this with a file that's in `:source-path` and that has underscores rather than hyphens in the name?

The commit message failure is because I require the body to be wrapped to 72 characters, not 80. My hesitation, though, is that I've bent over backwards to keep this...

You can use `fugitive#GitVersion(1, 7)` to check for a Git version with [`%N` support](https://github.com/git/git/commit/8b208f0213ed349ecb2ab8f7bb6c5072f8011a70).

I bring it up as a possible simplification because I haven't deciphered what `FuGiTiVeEnDmArKeR` is for but it's raising all my alarm bells.