Skip to content

ci: add examples to PR title validatior#1611

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
farisaziz12:ci/pr-title-validation
Feb 23, 2026
Merged

ci: add examples to PR title validatior#1611
danielroe merged 1 commit intonpmx-dev:mainfrom
farisaziz12:ci/pr-title-validation

Conversation

@farisaziz12
Copy link
Contributor

📚 Description

When opening this PR, I briefly encountered the PR validation failure before fixing it. Thought adding some examples could help quickly diagnose/correct for newcomers

@vercel
Copy link

vercel bot commented Feb 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 23, 2026 6:33pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 23, 2026 6:33pm
npmx-lunaria Ignored Ignored Feb 23, 2026 6:33pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8d040ad and eb4a9d2.

📒 Files selected for processing (1)
  • .github/workflows/semantic-pull-requests.yml

📝 Walkthrough

Walkthrough

The change modifies the .github/workflows/semantic-pull-requests.yml file to enhance the error message displayed when a pull request title's subject does not match the expected pattern. Specifically, the subjectPatternError message was augmented with an Examples block containing three pull request title examples: two valid examples (marked with checkmarks) and one invalid example (marked with a cross). Five lines were added and no lines were removed. This change affects the user-facing error messaging without altering any workflow logic or control flow.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly relates to the changeset, explaining the motivation for adding examples to the PR title validator error message.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielroe danielroe enabled auto-merge February 23, 2026 18:33
@danielroe danielroe added this pull request to the merge queue Feb 23, 2026
Merged via the queue into npmx-dev:main with commit 0fd5a7d Feb 23, 2026
16 checks passed
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

alex-key pushed a commit to alex-key/npmx.dev that referenced this pull request Feb 25, 2026
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