Merge pull request innogames#26 from innogames/feature/ignore-summary-length-for-merge-commits

kofrezo · web-flow · commit ab3f1245b6f0 · 2021-10-28T08:30:20.000+02:00
Reduce severity of length for merge commit summary
diff --git a/igcommit/commit_checks.py b/igcommit/commit_checks.py
@@ -137,7 +137,11 @@ def get_summary_problems(self, rest):
 
 rest_len = len(rest)
 if rest_len > 72:
- yield Severity.ERROR, 'commit summary longer than 72 characters'
+ if rest.startswith('Merge branch '):
+ sev = Severity.WARNING
+ else:
+ sev = Severity.ERROR
+ yield sev, 'commit summary longer than 72 characters'
 elif rest_len > 50:
 yield Severity.WARNING, 'commit summary longer than 50 characters'
 

-Original file line number
+Diff line change
 rest_len=len(rest)
 ifrest_len>72:
 -yieldSeverity.ERROR, 'commit summary longer than 72 characters'
 +ifrest.startswith('Merge branch '):
 +sev=Severity.WARNING
 +else:
 +sev=Severity.ERROR
 +yieldsev, 'commit summary longer than 72 characters'
 elifrest_len>50:
 yieldSeverity.WARNING, 'commit summary longer than 50 characters'