[question] should `skip_merge` also include `Merge pull request...` messages

[hamza-shezad]

when skip_merge is true, should commits with messages as following be skipped?

Merge pull request #111 from <repo>`

[robinmayerhofer]

Docs https://github.com/mergeability/mergeable/blob/590a40fa9ab9a5f3adbc11273687f043beb91aec/docs/validators/commit.rst#L10

and implementation https://github.com/mergeability/mergeable/blob/590a40fa9ab9a5f3adbc11273687f043beb91aec/lib/validators/commit.js#L69

      orderedCommits = orderedCommits.filter(commit => !commit.message.includes('Merge branch'))

diverge. Merge vs. Merge branch.

=> Fix the implementation?

[hamza-shezad]

yes. do you want me to open a PR?

[robinmayerhofer]

Yes, I would suggest that you open a PR and ping the maintainers!