David Barnett
David Barnett
Looks like #111 asked about the same, but nothing ever got added to the README. Unlike fzf, skim often wouldn't be installed through a system package manager, so having some...
I did start a guide at https://github.com/google/vim-codefmt/wiki/Formatter-Integration-Guide, but the parts you're probably interested in are just TODOs at the moment.
Yeah, per-path config is one concern (particularly w/ the `--aosp` example), and I have a few other minor concerns: they might need to override flags we're passing by default (like...
Github doesn't let me add you as an assignee, but we'll consider it assigned. =) What did you have in mind? I'd suggest starting with a script-local helper for the...
FWIW, goimports is more than a "formatter" and technically is not a "gofmt executable". If we run into a lot of other gotchas like this we may want to reconsider...
Another idea is to cd before running goimports (or whatever tool is configured in gofmt_executable), but it's a bit of a hack when there's already a `--srcdir` option supported and...
BTW, despite its name, `--srcdir` accepts the *full file path* including filename, not just the dir (at least [since 2016](https://golang.org/cl/25140)). We should make it invoke `goimports --srcdir=foo/bar.go` instead of `goimports...
I can include these fixes in with other test fixes I'm making. Apparently we had a few different unhandled breakages lately.
For "2. Consider a non-standard version to always have the feature" — That's just to say we have no idea, so we'll err on the side of trying to use...
@danishprakash Does it format? I see linting but can't find anything about transforming code in the README.