ci: optimize semantic PR title check workflow (@d1rshan)#7888
Open
d1rshan wants to merge 1 commit into
Open
Conversation
github-actions
Bot
added
the
waiting for review
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
Optimizes the “Semantic PR Title” GitHub Actions workflow to reduce redundant executions and unnecessary sticky-comment updates, while keeping the title validation rules unchanged.
Changes:
- Add concurrency to cancel older in-progress runs for the same PR.
- Skip
pull_request_target.editedruns when the PR title didn’t change. - Avoid rewriting the sticky error comment when its content is unchanged.
github-actions
Bot
removed
the
waiting for review
github-actions
Bot
added
the
waiting for review
d1rshan
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Optimizes the semantic PR title check workflow
Add concurrency group: Cancels any in-progress run when a newer one is triggered on the same PR, avoiding redundant runs when a title is edited multiple times in quick succession.
Switch to
ubuntu-slimrunner: This workflow only does lightweight title linting, so a fullubuntu-latestVM is overkill.ubuntu-slimis a GitHub-hosted single-CPU container runner optimized for short automation tasks like this, and is faster to spin up.Skip job on non-title edits: Added a condition to skip the job when a PR is edited but the title wasn't changed (e.g. body edits, label changes). Now the workflow would re-run on any edit event unnecessarily.
Add
skip_unchanged: trueto comment step: Avoids a redundant API call to update the sticky error comment if its content hasn't changed since the last run.