Skip to content

Document signed commits and merge-commit-only PRs - #163

Merged
GaryJones merged 1 commit into
developfrom
GaryJones/vipplug-173-review
Oct 9, 2026
Merged

GaryJones merged 1 commit into
developfrom
GaryJones/vipplug-173-review

Conversation

@GaryJones

Copy link
Copy Markdown
Contributor

Summary

The develop and main rulesets now reject unsigned commits, and the repository only allows merge commits (squash and rebase merging are both disabled). Neither was written down anywhere a contributor would see it, so the first anyone would learn of the signing requirement is a blocked merge button on an otherwise ready PR.

CONTRIBUTING.md now tells contributors to sign their commits and links to GitHub's guide, using the same wording as the other VIP plugin repos. AGENTS.md still told agents to squash and merge, which the repository no longer permits, so that line now describes the merge-commit workflow instead.

Refs VIPPLUG-173.

@GaryJones
GaryJones requested a review from a team as a code owner October 9, 2026 00:20
@GaryJones GaryJones added this to the Next milestone Oct 9, 2026
@GaryJones GaryJones added the type: documentation Documentation improvements label Oct 9, 2026
@GaryJones GaryJones self-assigned this Oct 9, 2026
Both rulesets now reject unsigned commits, so contributors need to know
before they open a pull request rather than discover it when the merge
button is blocked. The agent guidance still said to squash and merge,
which the repository no longer allows.

Refs VIPPLUG-173.
@GaryJones
GaryJones force-pushed the GaryJones/vipplug-173-review branch from 2574f49 to 574b2a3 Compare October 9, 2026 07:20
@GaryJones
GaryJones merged commit c2562da into develop Oct 9, 2026
7 checks passed
@GaryJones
GaryJones deleted the GaryJones/vipplug-173-review branch October 9, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: documentation Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant