"make check" should work

[jlevon]

Maybe dependent on #3 but "make check" in this repo should work.

[timfoster]

(that was overzealous ticket referencing by github - ignore it, it's not related)

[timfoster]

I ran prr through the joyent eslint plugins, which seemed to offer reasonable advice, and then through prettier/recommended, which imho made things look a bit worse, especially in the vasync pipelines:
https://gist.github.com/timfoster/b06c8f1ee75d494d72ace8b60759968b#file-prr-js-L898

Getting automatic code formatting is nice, but I don't think that's a price worth paying. I'll try a few of the other prettier configurations and see if those are less invasive, but suspect we'll just go with the joyent-eslint settings.

[timfoster]

With a few // prettier-ignore comments on the vasync blocks, what prettier produces isn't that bad, though it would take some getting used to. I also haven't tried to "re-prettier" any of the contents of the ignored blocks, which means we do have some inconsistency, which isn't wonderful, I admit :-/

I'll push the eslint and prettier fixes as separate commits to the PR to see if there's consensus on what we should do. My gut feeling is to drop prettier and just use our eslint rules.

[timfoster]

This is pull request #26