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: configure commitlint on CI #310

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Conversation

michelegera
Copy link
Contributor

@michelegera michelegera commented Nov 30, 2023

With this PR we want to build upon #309 and set up commitlint to run on CI when pushing to main or doing any kind of activity on a pull request.

This workflow will:

  • Check out the repository with all its history
  • Set up Node and pnpm
  • Install commitlint (with cache support)
  • Validate last commit and PR commits

@michelegera michelegera force-pushed the configure-commitlint-ci branch 5 times, most recently from 1475d5c to 16970dc Compare November 30, 2023 15:06
@michelegera michelegera changed the title Configure commitlint ci Configure commitlint on CI Nov 30, 2023
@michelegera michelegera changed the base branch from main to install-config-commitlint November 30, 2023 15:15
@michelegera michelegera changed the title Configure commitlint on CI chore: configure commitlint on CI Nov 30, 2023
@michelegera michelegera changed the base branch from install-config-commitlint to main November 30, 2023 15:15
@michelegera michelegera force-pushed the configure-commitlint-ci branch from 16970dc to 96b7e02 Compare November 30, 2023 15:22
@michelegera michelegera force-pushed the configure-commitlint-ci branch from e283d98 to 6158f7c Compare December 4, 2023 14:06
@michelegera michelegera marked this pull request as ready for review December 4, 2023 14:07
@michelegera michelegera merged commit 1b51058 into main Dec 5, 2023
17 checks passed
@michelegera michelegera deleted the configure-commitlint-ci branch December 5, 2023 11:30
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.

2 participants