git subrepo pull (merge) external/os-autoinst-common

subrepo: subdir: "external/os-autoinst-common" merged: "9e7ec164e" upstream: origin: "[email protected]:os-autoinst/os-autoinst-common.git" branch: "master" commit: "1b05ad55d" git-subrepo: version: "0.4.6" origin: "???" commit: "???"
os-autoinst · Oct 24, 2023 · 7ad022a · 7ad022a
1 parent c944acc
commit 7ad022a
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 8 deletions.
diff --git a/external/os-autoinst-common/.github/workflows/commit_message_checker.yml b/external/os-autoinst-common/.github/workflows/commit_message_checker.yml
@@ -1,25 +1,29 @@
 ---
 # https://github.com/marketplace/actions/gs-commit-message-checker
-name: 'Commit Message Check'
+name: 'Commit message check'
 # yamllint disable-line rule:truthy
-on: [push, pull_request]
+on:
+  pull_request:
+  push:
+    branches:
+      - '!master'  # we must not fix commit messages when they already reached master
 
 jobs:
   check-commit-message:
-    name: Check Commit Message
+    name: Check commit message
     runs-on: ubuntu-latest
     steps:
-      - name: Check Subject Begining
+      - name: Check subject beginning
         uses: gsactions/commit-message-checker@v1
         with:
-          pattern: '^([A-Z]|\S+:)'
+          pattern: '^([A-Z]|[A-Za-z0-9_/.\-\s]+:|git subrepo pull)'
           flags: 'g'
           error: 'The subject does not start with a capital or tag.'
           excludeDescription: 'true'
           excludeTitle: 'true'
           checkAllCommitMessages: 'true'
           accessToken: ${{ secrets.GITHUB_TOKEN }}
-      - name: Check Subject Line Length
+      - name: Check subject line length
         uses: gsactions/commit-message-checker@v1
         with:
           pattern: '^.{1,72}(\n|$)'
@@ -29,7 +33,17 @@ jobs:
           excludeTitle: 'true'    # excludes the title of a pull request
           checkAllCommitMessages: 'true'    # checks all commits associated with a pull request
           accessToken: ${{ secrets.GITHUB_TOKEN }}    # only required if checkAllCommitMessages is true
-      - name: Check Empty Line
+      - name: Check subject ending
+        uses: gsactions/commit-message-checker@v2
+        with:
+          pattern: '^.+(?<!\.)(\n|$)'
+          flags: 'g'
+          error: 'The subject cannot end with a dot.'
+          excludeDescription: 'true'
+          excludeTitle: 'true'
+          checkAllCommitMessages: 'true'
+          accessToken: ${{ secrets.GITHUB_TOKEN }}
+      - name: Check empty line
         uses: gsactions/commit-message-checker@v1
         with:
           pattern: '^.*(\n\n|$)'

diff --git a/external/os-autoinst-common/.gitrepo b/external/os-autoinst-common/.gitrepo
@@ -6,7 +6,7 @@
 [subrepo]
 	remote = [email protected]:os-autoinst/os-autoinst-common.git
 	branch = master
-	commit = e555e723f0e784d1973b5f141a5ec4a432a71b9e
+	commit = 1b05ad55dabef8176f683f24c5e70283a6ca2135
 	parent = 98e54f9632959aed6a15e3d0afe0a9bdb6249bd4
 	method = merge
 	cmdver = 0.4.6