Skip to content

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.

4 participants