Add perl formatting support using Perltidy#227
Add perl formatting support using Perltidy#227liuyehcf wants to merge 2 commits intogoogle:masterfrom
Conversation
|
Sorry, I think since the tests weren't passing this fell through the cracks. I think if you can rebase your changes on top of the latest upstream and address my comment there to fix the tests, this looks ready to merge otherwise. |
| must be installed on your system. (But not for testing; vroom intercepts | ||
| system calls.) | ||
| :FormatCode perltidy | ||
| ! perltidy .* |
There was a problem hiding this comment.
You need to fake the command output like this:
vim-codefmt/vroom/clangformat.vroom
Line 46 in c6730fb
Otherwise the tests can't run without perltidy installed (and may get brittle, failing depending on the version of perltidy installed).
There was a problem hiding this comment.
Updated, and I'm not sure if I'm doing right because I don't really understand the syntax of .vroom.
Signed-off-by: liuyehcf <1559500551@qq.com>
Signed-off-by: liuyehcf <liuyehcf@gmail.com>
|
I can help update the tests if a current maintainer can review. @malcolmr @fowles @snu5mumr1k |
|
Here ya go, patch dbarnett@de65871 and you should be good to go. |
2 participants
Solves #210