Build: Remove commit check from appveyor (fixes #6292)

[alberto]

No description provided.

[eslintbot]

LGTM

[mention-bot]

By analyzing the blame information on this pull request, we identified @IanVS, @nzakas and @gyandeeps to be potential reviewers

[eslintbot]

LGTM

[IanVS]

👍

[IanVS]

Actually, shouldn't it also be removed from package.json and makefile.js? It's essentially dead code at this point, isn't it?

[alberto]

Mmm, now that you mention it... @nzakas did you actually intend to remove all git checks or just the number of commits in #6263? check-commit also validates the commit message format and length.

[nzakas]

The bot already does the other checks, so we don't need Travis/Appveyor doing it as well.

I left the task in Makefile.js because I was unsure if developers were using it locally (it is mentioned in the pull request docs).

[platinumazure]

@nzakas I sometimes use it locally. That said, I think we can leave it in Makefile.js without actually invoking it in npm test or in Travis/AppVeyor.

[alberto]

Ok, makes sense. PR is good as is, then.