Skip to content

feat: adds action to lint conventional commits #1902

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

Merged
merged 3 commits into from
Mar 11, 2025

Conversation

karangattu
Copy link
Collaborator

This pull request introduces a new GitHub Actions workflow to enforce the use of Conventional Commits for pull request titles. The workflow includes steps to lint the pull request title and provide feedback if it does not comply with the Conventional Commits specification.

@karangattu karangattu requested a review from schloerke March 11, 2025 18:11
@schloerke schloerke changed the title ci: adds action to lint conventional commits not-valid: adds action to lint conventional commits Mar 11, 2025
@schloerke schloerke enabled auto-merge (squash) March 11, 2025 18:28
@schloerke schloerke changed the title not-valid: adds action to lint conventional commits feat: adds action to lint conventional commits Mar 11, 2025
Comment on lines +37 to +41
Details are below the break:

----------------------------

${{ steps.lint.outputs.error_message }}
Copy link
Collaborator

@gadenbuie gadenbuie Mar 11, 2025

Choose a reason for hiding this comment

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

Can we put this in a details element instead?

Suggested change
Details are below the break:
----------------------------
${{ steps.lint.outputs.error_message }}
<details><summary>Additional details</summary>
${{ steps.lint.outputs.error_message }}
</details>

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll PR this

Copy link
Collaborator

Choose a reason for hiding this comment

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

Those tags hide the information by default. If we can have it auto expanded, I'd be up for it.

@schloerke schloerke merged commit 69dccd6 into main Mar 11, 2025
42 of 43 checks passed
@schloerke schloerke deleted the add-lint-conventional-commits branch March 11, 2025 18:41
schloerke added a commit that referenced this pull request Mar 11, 2025
* main:
  chore: Use details for extra info from error message (#1903)
  ci: adds action to lint conventional commits (#1902)
  build: JS build script updates (#1894)
  chore: Rename temp environ names (for launching browser) to avoid conflict with local environ names (#1899)
  build: Update shiny TS types to Shiny 1.10.0 (#1893)
  chore: Use `shiny.module` to import re-exported values from `shiny._namespaces` (#1898)
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.

3 participants