Skip to content

feat: Skip lint when merging remote branch#207

Merged
folke merged 1 commit intofolke:masterfrom
singara-velan:feature/lint-allow-remote-branch-mege-commit-message
Oct 20, 2022
Merged

feat: Skip lint when merging remote branch#207
folke merged 1 commit intofolke:masterfrom
singara-velan:feature/lint-allow-remote-branch-mege-commit-message

Conversation

@singara-velan
Copy link
Copy Markdown
Contributor

Description

When merging a remote branch (origin/master) directly into the local branch (feature/branch-abc), the default git commits message will be "Merge remote-tracking branch 'origin/master' into feature/branch-abc".

Current lint doesn't cover this use case, so the commit message format is validated against the conventional commit standards and getting blocked.

This PR includes a regex to skip the git merge default commit message from linting.

@singara-velan singara-velan marked this pull request as ready for review June 15, 2022 13:32
@singara-velan
Copy link
Copy Markdown
Contributor Author

Dear @folke, we are actively using devmoji in our project. Please review the PR and let me know of any changes needed.
Looking forward to merging the PR when possible :) .

@folke folke merged commit 55bbef8 into folke:master Oct 20, 2022
github-actions bot pushed a commit that referenced this pull request Oct 20, 2022
## [2.4.0](v2.3.2...v2.4.0) (2022-10-20)

### Features

* Skip lint when merging remote branch ([#207](#207)) ([55bbef8](55bbef8))

### Other

* **deps:** update dependency conventional-changelog-conventionalcommits to v4.6.3 ([ef024ef](ef024ef))
* **deps:** update dependency minimist to v1.2.6 [security] ([9e7ecb7](9e7ecb7))
* **deps:** update dependency node-fetch to v2.6.6 ([23d00ce](23d00ce))
* **deps:** update dependency node-fetch to v2.6.7 [security] ([3239b8c](3239b8c))
* **deps:** update dependency ts-interface-builder to v0.3.3 ([7c6392f](7c6392f))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants