Skip to content

[FEATURE] Skip merge commits in pull requests #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

[FEATURE] Skip merge commits in pull requests #79

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
fe2e809
[FEATURE] Skip merge commits in pull requests
Aug 27, 2022
403524f
[TASK] Extend test
gilbertsoft Oct 20, 2022
8c35571
[TASK] Update dist
gilbertsoft Oct 20, 2022
2f931d9
[TASK] Ignore merge commits for push event
gilbertsoft Oct 20, 2022
60cf4b8
[TASK] Shorten filter function
gilbertsoft Oct 20, 2022
42ac016
[BUGFIX] Ignore merge commits for push events
gilbertsoft Oct 20, 2022
e744789
[TASK] Add excludeMergeCommits option
gilbertsoft Oct 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ More information about `pattern` and `flags` can be found in the

`flags` is optional and defaults to `gm`.

`excludeDescription`, `excludeTitle` and `checkAllCommitMessages` are optional.
Default behavior is to include the description and title and not check pull
request commit messages.
`excludeDescription`, `excludeTitle`, `excludeMergeCommits`, and
`checkAllCommitMessages` are optional. Default behavior is to include the
description, title, and merge commits, and not check pull request commit messages.

### Example Workflow

Expand Down Expand Up @@ -67,8 +67,9 @@ jobs:
error: 'The maximum line length of 74 characters is exceeded.'
excludeDescription: 'true' # optional: this excludes the description body of a pull request
excludeTitle: 'true' # optional: this excludes the title of a pull request
excludeMergeCommits: 'true' # optional: this excludes merge commits
checkAllCommitMessages: 'true' # optional: this checks all commits associated with a pull request
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages is true
accessToken: ${{ secrets.GITHUB_TOKEN }} # github access token is only required if checkAllCommitMessages or excludeMergeCommits is true
- name: Check for Resolves / Fixes
uses: gsactions/commit-message-checker@v2
with:
Expand Down
2 changes: 1 addition & 1 deletion __tests__/commit-message-checker.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/*
* This file is part of the "GS Commit Message Checker" Action for Github.
*
* Copyright (C) 2019 by Gilbertsoft LLC (gilbertsoft.org)
* Copyright (C) 2019-2022 by Gilbertsoft LLC (gilbertsoft.org)
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
Expand Down
Loading