Skip to content

Commit d4519d4

Browse files
arunkannawadiarunkannawadi
committed
Leave extra space before comments
1 parent 99b5990 commit d4519d4

1 file changed

Lines changed: 34 additions & 34 deletions

File tree

‎.github/workflows/commit-checker.yaml‎

Lines changed: 34 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -15,60 +15,60 @@ jobs:
1515
- name: Check Subject Line Length
1616
uses: gsactions/commit-message-checker@v2
1717
with:
18-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
19-
excludeTitle: 'true' # optional: this excludes the title of a pull request
20-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
21-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
18+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
19+
excludeTitle: 'true' # optional: this excludes the title of a pull request
20+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
21+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
2222
pattern: '^.{0,50}(\n.*)*$'
23-
error: 'Subject too long (max 50)'
23+
error: 'Subject line was too long, exceeding 50 characters. Continue in the body of the commit message'
2424
- name: Check Subject Line Capitalization
2525
if: ${{ success() || failure() }}
2626
uses: gsactions/commit-message-checker@v2
2727
with:
28-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
29-
excludeTitle: 'true' # optional: this excludes the title of a pull request
30-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
31-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
28+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
29+
excludeTitle: 'true' # optional: this excludes the title of a pull request
30+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
31+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
3232
pattern: '^[A-Z]'
33-
error: 'Subject line must begin with an imperative, capitalized'
33+
error: 'Subject line must begin with an imperative verb in present tense and first letter in uppercase'
3434
- name: Check Subject Line Ending
3535
if: ${{ success() || failure() }}
3636
uses: gsactions/commit-message-checker@v2
3737
with:
38-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
39-
excludeTitle: 'true' # optional: this excludes the title of a pull request
40-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
41-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
42-
pattern: '(?<!\.)$'
43-
error: 'Subject line must not end with a period'
38+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
39+
excludeTitle: 'true' # optional: this excludes the title of a pull request
40+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
41+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
42+
pattern: '(?<![\.\?\!])$'
43+
error: 'Subject line must not end with a period, question mark or exclamation!'
4444
- name: Check Body Line Length
4545
if: ${{ success() || failure() }}
4646
uses: gsactions/commit-message-checker@v2
4747
with:
48-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
49-
excludeTitle: 'true' # optional: this excludes the title of a pull request
50-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
51-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
52-
pattern: '^.+(\n.{0,72}\n)*$'
53-
error: 'Body line too long (max 72)'
54-
- name: Check that main was not merged
48+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
49+
excludeTitle: 'true' # optional: this excludes the title of a pull request
50+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
51+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
52+
pattern: '^.+((\n*.{0,72}\n)*)*$'
53+
error: 'The body must not exceed 72 characters per line and must have end with a newline character.'
54+
- name: Check body ends with an appropriate punctuation (. or ? or !)
5555
if: ${{ success() || failure() }}
5656
uses: gsactions/commit-message-checker@v2
5757
with:
58-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
59-
excludeTitle: 'true' # optional: this excludes the title of a pull request
60-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
61-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
62-
pattern: ^(?!Merge branch 'main' into)
63-
error: 'Ticket branch needs to be rebased onto main'
58+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
59+
excludeTitle: 'true' # optional: this excludes the title of a pull request
60+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
61+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
62+
pattern: '^.+(\n*(.+[\.\!\?])*)*$'
63+
error: 'The text of the body must end with a period, question mark or exclamation!'
6464
# Check the full pattern, now that we covered more common mistakes.
6565
- name: Check overall commit Message
66-
if: ${{ success() || failure() }}
66+
if: ${{ success() }}
6767
uses: gsactions/commit-message-checker@v2
6868
with:
69-
excludeDescription: 'true' # optional: this excludes the description body of a pull request
70-
excludeTitle: 'true' # optional: this excludes the title of a pull request
71-
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
72-
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
69+
excludeDescription: 'true' # optional: this excludes the description body of a pull request
70+
excludeTitle: 'true' # optional: this excludes the title of a pull request
71+
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
72+
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
7373
pattern: '^(?![A-Z].*\.)[A-Z].{0,49}(\r?|\r?\n(?:.{0,71}\r?\n)*(?:.{0,71}\.)?\r?\n?)$'
7474
error: "The commit messages do not follow "

0 commit comments

Comments
 (0)