Skip to content

feat: add github action lint for PR titles - #89

Merged
wiktoriavh merged 4 commits into
r-webdev:mainfrom
michal-skraburski:github-action
Aug 27, 2026
Merged

feat: add github action lint for PR titles#89
wiktoriavh merged 4 commits into
r-webdev:mainfrom
michal-skraburski:github-action

Conversation

@michal-skraburski

Copy link
Copy Markdown
Contributor

This PR is made to satisfy issue #88

Following the conventions from
https://www.conventionalcommits.org/en/v1.0.0/#summary

It was decided to reuse https://github.com/marketplace/actions/semantic-pull-request as this action seemed far more solid of answer than implementing a custom version which may or may not work. This marketplace option is a good choice because it is tried and tested by many projects.

Comment thread .github/workflows/pr-check.yml Outdated
Comment thread .github/workflows/pr-check.yml Outdated
Comment thread .github/workflows/pr-check.yml Outdated
michal-skraburski and others added 3 commits August 25, 2026 08:22
Co-authored-by: Wiktoria van Harneveldt <62669899+wiktoriavh@users.noreply.github.com>
Co-authored-by: Wiktoria van Harneveldt <62669899+wiktoriavh@users.noreply.github.com>

@michal-skraburski michal-skraburski left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just finished updating the requests and verified there's few downsides to enabling the synchro option. It does seem recommended to make it on, why that isn't the default is confusing to me, but now it's enabled.

Do note this feature will use up more of your GitHub Actions plan minutes, and changing the PR title is not at all common, especially in small repos like this one. :S @wiktoriavh

@wiktoriavh

wiktoriavh commented Aug 27, 2026

Copy link
Copy Markdown
Member

Do note this feature will use up more of your GitHub Actions plan minutes,

that's alright. we don not do enough work that we will run into an issue here.

and changing the PR title is not at all common,

what do you mean by that?

@wiktoriavh
wiktoriavh merged commit d3d8f8a into r-webdev:main Aug 27, 2026
1 check passed
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.

Add action that will make the pipeline red if the PR title does not follow the schema

2 participants