Skip to content

feat(pr-title): Add check for title length#121

Merged
chris3ware merged 6 commits intomainfrom
ci-pr-title
Feb 13, 2025
Merged

feat(pr-title): Add check for title length#121
chris3ware merged 6 commits intomainfrom
ci-pr-title

Conversation

@chris3ware
Copy link
Copy Markdown
Member

This repository uses 'Squash & Merge' with a default commit message of 'Pull Request Title' when merging pull requests. Most good commit message guides suggest a commit message length of 72. A check has been added to ensure the length of the pull request title does not exceed 72 characters.

Removed unnecessary quotes around the failure summary expression
because they were causing the step to fail.
Use the functionality available in the existing job to check the
length of the PR title and enforce sentence case for the subject.
@chris3ware
Copy link
Copy Markdown
Member Author

See #124 for testing of the new pr workflow.

@chris3ware chris3ware marked this pull request as ready for review February 13, 2025 09:37
@chris3ware chris3ware requested a review from a team as a code owner February 13, 2025 09:37
@chris3ware chris3ware merged commit 4e7167e into main Feb 13, 2025
@chris3ware chris3ware deleted the ci-pr-title branch February 13, 2025 09:41
3ware-release Bot pushed a commit that referenced this pull request Feb 13, 2025
## [4.6.0](v4.5.1...v4.6.0) (2025-02-13)

### Features

* **pr-title:** Add check for title length ([#121](#121)) ([4e7167e](4e7167e))
@3ware-release
Copy link
Copy Markdown
Contributor

3ware-release Bot commented Feb 13, 2025

This PR is included in version 4.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants