Don't do commit message checks on merges? about grumphp

[veewee]

That could be a nice addition to the task.
There is some sense of merge commit detection already, but we could add a flag to skip the task on merge commits.
Care to give it a try?

[veewee]

The code for this specific task is located here:
https://github.com/phpro/grumphp/blob/v2.x/src/Task/Git/CommitMessage.php

Currently, there is a merge matching mechanism for skipping validation on scope conventions:

grumphp/src/Task/Git/CommitMessage.php

Lines 366 to 384 in d0e23de

	$mergePattern =
	'(Merge branch|tag \'.+\'(?:\s.+)?|Merge remote-tracking branch \'.+\'|Merge pull request #\d+\s.+)';
	if (count($types) > 0) {
	$types = implode('|', $types);
	$typesPattern = '(' . $types . ')';
	}
	if (count($scopes) > 0) {
	$scopes = implode('|', $scopes);
	$scopesPattern = '(:\s|(\((?:' . $scopes . ')\)?:\s))';
	}
	$rule = '/^' . $specialPrefix . $typesPattern . $scopesPattern . $subjectPattern . '|' . $mergePattern . '/';
	try {
	$this->runMatcher($config, $subjectLine, $rule, 'Invalid Type/Scope Format');
	} catch (RuntimeException $e) {
	throw $e;
	}

It might make sense to extract these merge patterns into a separate function so that it can be used to determine if the task needs to run at all. You could use that same check to skip running the scope conventions.

That specific task is becoming a bit messy because of historical reasons and is open for improvements :)

[delboy1978uk]

@veewee sure, can you point me to the code I should be looking at?

[delboy1978uk]

Thanks @veewee I'll have a look!
BTW I see you're in Belgium, me too! (ik woon al 7 jaar in Leuven)

[veewee]

Nice :) (Ik ben van de Kempen!)

[delboy1978uk]

I'm now able to merge without a JIRA number on a merge! But let me know if I should move that is merge commit check further up or down the chain of events, I allowed it to still go through the first rules, like not empty message etc. Also, you mentioned a flag setting so we can have the merge skip enabled or disabled, how/where do we go about adding the setting?

[delboy1978uk]

Of course, I have broken your tests in doing this :P But I'll wait for your feedback first before continuing work!

[veewee]

Cool. Let's continue the discussion in the PR.