Skip to content

Ci/234 pr validation#259

Merged
phertyameen merged 1 commit intoMindBlockLabs:mainfrom
mijinummi:ci/234-pr-validation
Feb 24, 2026
Merged

Ci/234 pr validation#259
phertyameen merged 1 commit intoMindBlockLabs:mainfrom
mijinummi:ci/234-pr-validation

Conversation

@mijinummi
Copy link
Contributor

Enforce PR Title Convention & Description Requirements (#234)

Problem

  • Contributors submit PRs with vague titles (update) or minimal descriptions (Closes #22).
  • This slows reviews and pollutes commit history.

Solution

  • Added validate-pr job in .github/workflows/ci-cd.yml.
  • Updated build-and-deploy job to depend on validation.
  • Updated CONTRIBUTING.md with clear PR standards.

Acceptance Criteria

  • ✅ PR titles follow Conventional Commits style.
  • ✅ PR descriptions provide context (≥20 chars).
  • ✅ CI blocks non-compliant PRs.
  • ✅ Contributors informed via updated guidelines.

Testing

Copy link
Member

@phertyameen phertyameen left a comment

Choose a reason for hiding this comment

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

Make a pull request please. Some important changes were made. these checks oughtt to pass immediately after

@mijinummi mijinummi force-pushed the ci/234-pr-validation branch from 19e52d6 to 585d623 Compare February 24, 2026 09:08
@phertyameen phertyameen merged commit c9d3812 into MindBlockLabs:main Feb 24, 2026
3 checks passed
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.

Enforce PR Title Convention & Description Requirements in CI/CD Pipeline

2 participants