Skip to content

Add the action for PR Metadata validation#784

Draft
j-rivero wants to merge 4 commits into
mainfrom
jrivero/pr-validation-v2
Draft

Add the action for PR Metadata validation#784
j-rivero wants to merge 4 commits into
mainfrom
jrivero/pr-validation-v2

Conversation

@j-rivero
Copy link
Copy Markdown
Contributor

@j-rivero j-rivero commented Jan 20, 2026

🎉 New feature

Related and a resurrection of gazebo-tooling/release-tools#246. Follow up of #670

Summary

Second integration of the https://github.com/gazebo-tooling/pr-data-action/ into the main branch only of this repository. The github action added will check each of the PRs target to main for the presence of a changelog entry inside the .changelog directory following conventional commits.

Testing

Testing validation workflow iteratively by triggering different error states.

@j-rivero j-rivero requested a review from caguero as a code owner January 20, 2026 17:48
@j-rivero j-rivero changed the base branch from gz-transport15 to main January 20, 2026 17:49
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Changelog Validation Failed

The following issues were found with this PR:

  • Missing changelog file
    • Please add a changelog file to the .changelog/ directory
    • Name it descriptively (e.g., fix-bug-123.md, add-new-feature.md)
    • The content must follow conventional commits format

Please fix the issues above and push your changes. The validation will run again automatically.

@github-project-automation github-project-automation Bot moved this from Inbox to In review in Core development Jan 20, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Changelog Validation Failed

The following issues were found with this PR:

  • Changelog file found

  • Invalid changelog content format - The changelog content must follow the Conventional Commits specification

Error:

Error: Missing commit type separator `:`

Caused by:
     --> 1:5
      |
    1 | This is not a conventional commit format
      |     ^---
      |
      = expected scope or type_separator

Valid format examples:

feat: add flying robot support
fix: fix a memory link in the walking algorithm

Please fix the issues above and push your changes. The validation will run again automatically.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Changelog Validation Passed

All required data for the changelog validation checks have passed:

  • Changelog file found - Changes are documented
  • Content format valid - Follows conventional commits specification

This PR is ready for review! 🚀

@j-rivero j-rivero marked this pull request as draft January 20, 2026 19:10
@azeey azeey moved this from In review to In progress in Core development May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

2 participants