Skip to content
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.

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

chore: enforce semantic PR title #7

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

jeanregisser
Copy link
Member

@jeanregisser jeanregisser commented Oct 26, 2022

Now that we're using semantic-release it's important PR titles follow the convention.

https://github.com/marketplace/actions/semantic-pull-request is a simple solution for this.

We should probably add it to https://github.com/valora-inc/typescript-app-starter too.

Copy link

@jacob-israel-turner jacob-israel-turner left a comment

Choose a reason for hiding this comment

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

Awesome! 🎉

@jeanregisser jeanregisser merged commit 81f20f3 into main Oct 26, 2022
@jeanregisser jeanregisser deleted the jeanregisser/enforce-conventional-pr-title branch October 26, 2022 13:24
@github-actions
Copy link

🎉 This PR is included in version 1.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

jeanregisser added a commit to valora-inc/typescript-app-starter that referenced this pull request Dec 5, 2022
We decided to use semantic PR title as it can help with automated
releases and changelog generation.
This helps ensure new repos will enforce that rule too.

Already used in valora-inc/logging#7

See also:
https://valora-app.slack.com/archives/C025V1D6F3J/p1669991436336709
jeanregisser added a commit to valora-inc/wallet that referenced this pull request Jan 3, 2023
### Description

We decided to use semantic PR title as it can help with automated
releases and changelog generation.

Already used in valora-inc/logging#7 and
valora-inc/typescript-app-starter#51

### Test plan

N/A

### Related issues

Discussed on
[Slack](https://valora-app.slack.com/archives/C025V1D6F3J/p1669991436336709).

### Backwards compatibility

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants