Skip to content

chore: only apply commit convention to PR titles#11296

Merged
patak-cat merged 4 commits intomainfrom
drop-commit-check
Dec 14, 2022
Merged

chore: only apply commit convention to PR titles#11296
patak-cat merged 4 commits intomainfrom
drop-commit-check

Conversation

@ArnaudBarre
Copy link
Copy Markdown
Member

As proposed in Discord, almost all commits on main comes from GH PRs, and adding a validation to every commit of any branch creates a lot of friction for nothing IMO.

jeffydc
jeffydc previously approved these changes Dec 10, 2022
benmccann
benmccann previously approved these changes Dec 12, 2022
Comment thread .github/workflows/semantic-pull-request.yml Outdated
Co-authored-by: patak <matias.capeletto@gmail.com>
@ArnaudBarre ArnaudBarre dismissed stale reviews from benmccann and jeffydc via 0551531 December 12, 2022 19:09
@ArnaudBarre
Copy link
Copy Markdown
Member Author

Just saw that there is semantic.yml file, I will merge both options in the workflow file

@ArnaudBarre
Copy link
Copy Markdown
Member Author

@benmccann The file you added in #4424 seems outdated since #7826

I already updated the readme to be in sync with the default of the action. Bot's PR are using chore(deps) so I think that's fine to not have deps type (plus it will simpler to fix the changelog generation if most non-relevant commit for users are under chore)

@patak-cat
Copy link
Copy Markdown
Member

Maybe we should leave the file named commit-convention.md instead of pr-title-convention.md. It is still a commit convention, but we only apply it to our main and release branches. I think folks are more used to the old name.

@patak-cat patak-cat enabled auto-merge (squash) December 14, 2022 21:04
@patak-cat patak-cat merged commit 27362cb into main Dec 14, 2022
@patak-cat patak-cat deleted the drop-commit-check branch December 14, 2022 21:12
futurGH pushed a commit to futurGH/vite that referenced this pull request Feb 26, 2023
sxzz added a commit to vitejs/vite-plugin-vue that referenced this pull request Aug 28, 2023
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.

4 participants