search

Vimjas / vim-testbed

Docker image for testing Vim plugins
42 stars 6 forks source link

install_vim: fix some issues reported by shellcheck #3

Closed blueyed closed 8 years ago

tweekmonster commented 8 years ago

Thanks! I guess shellcheck should be included in the travis build as well.

blueyed commented 8 years ago

I guess shellcheck should be included in the travis build as well.

It is more for linting shell scripts, and not Vim scripts. But vint (or vimlint) might be useful to have by default.

tweekmonster commented 8 years ago

It is more for linting shell scripts, and not Vim scripts.

I was thinking it'd be for checking PRs. OTOH, this repo should be fairly low maintenance.

But vint (or vimlint) might be useful to have by default.

Do you mean as part of the image?

blueyed commented 8 years ago

I was thinking it'd be for checking PRs.

Ah, makes sense.

Do you mean as part of the image?

Yes.