Skip check for commits from bots

[bkeepers]

#20 has a commit from a bot, which fails the DCO check. Any commit where the author name includes [bot] should be ignored.

[hiimbex]

Also similar scenario where the bot is an OAuth actor? I assume the problem would be the same there.

cc/ @tbarn

[tunnckoCore]

Hm. Good point.

[tunnckoCore]

I think one check in lib/dco.js for

(commit.author.type === 'Bot' && commit.author.login.endsWith('[bot]')) ||
commit.commiter.type === 'Bot' 

would be enough?

refs:
https://api.github.com/repos/tunnckoCore/parse-function/pulls/49
https://api.github.com/repos/tunnckoCore/parse-function/pulls/49/commits

Be aware of that, when Web UI is used, the commit.commiter.login is web-flow, so most guaranteed is commit.author.

[hiimbex]

sounds about right to me! Wanna open a PR for that @charlike?

[tunnckoCore]

Yea, i'll give it a shot when have a bit more time :)

[stale]

Is this still relevant? If so, please comment with any updates or addition details.