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

fix(ci): Skip the title check if handling a merge group #790

Merged
merged 2 commits into from
Feb 9, 2023

Conversation

phated
Copy link
Contributor

@phated phated commented Feb 9, 2023

Related issue(s)

Resolves #

Description

This updates the conventional commit workflow to check when something is added to a merge queue; however, it just skips the check step because we've already had it successful on the PR (and that title is being used for the merge queue).

Summary of changes

Dependency additions / changes

Test additions / changes

Checklist

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt with default settings.
  • I have linked this PR to the issue(s) that it resolves.
  • I have reviewed the changes on GitHub, line by line.
  • I have ensured all changes are covered in the description.
  • This PR requires documentation updates when merged.

Additional context

@phated phated requested a review from kevaundray February 9, 2023 17:24
@kevaundray kevaundray enabled auto-merge February 9, 2023 17:39
@kevaundray kevaundray disabled auto-merge February 9, 2023 18:15
@phated phated added this pull request to the merge queue Feb 9, 2023
@phated phated removed this pull request from the merge queue due to a manual request Feb 9, 2023
@phated phated added this pull request to the merge queue Feb 9, 2023
@kevaundray kevaundray removed this pull request from the merge queue due to a manual request Feb 9, 2023
@kevaundray kevaundray merged commit 71b179c into master Feb 9, 2023
@kevaundray kevaundray deleted the phated/mq-workflow-fix branch February 9, 2023 20:19
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