Skip to content

Conversation

@christian-schilling
Copy link
Member

Change: single-commit-pr

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new GitHub Actions workflow that enforces pull request quality standards by requiring PRs to contain exactly one commit and be up-to-date with the base branch.

Key Changes:

  • Added automated PR validation workflow that checks commit count and branch synchronization status
  • Implemented git-based ahead/behind calculation to ensure PRs have exactly 1 commit ahead and 0 commits behind the base branch
  • Configured workflow to run on PR open, synchronize, reopen, and ready_for_review events

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch 5 times, most recently from d8aa796 to b6556a2 Compare November 2, 2025 15:16
Copy link
Collaborator

@vlad-ivanov-name vlad-ivanov-name left a comment

Choose a reason for hiding this comment

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

lgtm but note this will require proper PR stacking -- multi-commit PRs that josh split push creates will fail CI

@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch from b6556a2 to c70a885 Compare November 2, 2025 16:24
@christian-schilling christian-schilling marked this pull request as draft November 2, 2025 16:31
@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch from c70a885 to aa305d4 Compare November 3, 2025 08:50
@christian-schilling christian-schilling marked this pull request as ready for review November 3, 2025 08:51
@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch 5 times, most recently from 3e6280e to 603448d Compare November 3, 2025 09:12
@christian-schilling christian-schilling marked this pull request as draft November 3, 2025 11:48
@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch 10 times, most recently from b8715ae to adefa63 Compare November 8, 2025 11:33
@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch 25 times, most recently from f1c4b86 to f797ac4 Compare November 13, 2025 17:47
@christian-schilling christian-schilling force-pushed the @changes/master/christian.schilling.de@gmail.com/single-commit-pr branch from f797ac4 to 6c8a676 Compare November 13, 2025 18:35
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.

3 participants