Skip to content

Conversation

@mhanberg
Copy link
Member

@mhanberg mhanberg commented Jul 3, 2025

Since we are going to use release-please to manage our releases and
changelog, we need to start using conventional commits and the Squash
and Merge style of PR merging.

This workflow will lint the pull request title to ensure that it follows
conventional commits, since the title will be used as the commit subject
when squash and merging

Since we are going to use release-please to manage our releases and
changelog, we need to start using conventional commits and the Squash
and Merge style of PR merging.

This workflow will lint the pull request title to ensure that it follows
conventional commits, since the title will be used as the commit subject
when squash and merging
@mhanberg
Copy link
Member Author

mhanberg commented Jul 3, 2025

Since this uses pull_request_target, I believe it won't actually run until its is merged and lives on main.

Copy link
Collaborator

@doorgan doorgan left a comment

Choose a reason for hiding this comment

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

Comparing with the next-ls ci, this looks like it should work

@mhanberg mhanberg merged commit 62e5515 into main Jul 3, 2025
12 checks passed
@mhanberg mhanberg deleted the ci-lint-commit branch July 3, 2025 04:39
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.

3 participants