Skip to content

Require PR metadata for changelog entries - #51

Merged
mavam merged 1 commit into
mainfrom
topic/require-pr-metadata-validation
Jul 3, 2026
Merged

Require PR metadata for changelog entries#51
mavam merged 1 commit into
mainfrom
topic/require-pr-metadata-validation

Conversation

@mavam

@mavam mavam commented Jul 3, 2026

Copy link
Copy Markdown
Member

🔍 Problem

  • Changelog entries can still land without PR metadata in this repository.
  • Local pre-push validation needs to tolerate missing PR numbers before the first push creates a pull request.

🛠️ Solution

  • Enable require_pr: true for the changelog project.
  • Keep CI changelog validation strict.
  • Run Lefthook changelog validation with --lenient for local pre-push checks.

💬 Review

  • Release-process-only change; no product changelog entry.
📎 Related: tenzir/ship#34

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@mavam
mavam force-pushed the topic/require-pr-metadata-validation branch from f2947e2 to 4fd0897 Compare July 3, 2026 10:50
Set changelog validation to require PR metadata. Keep CI validation strict while letting local pre-push hooks use lenient mode before a pull request number exists.

Assisted-by: ChatGPT (pi)
@mavam
mavam force-pushed the topic/require-pr-metadata-validation branch from 4fd0897 to 0a0bfac Compare July 3, 2026 10:56
@mavam
mavam merged commit 1ee3376 into main Jul 3, 2026
5 checks passed
@mavam
mavam deleted the topic/require-pr-metadata-validation branch July 3, 2026 15:13
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.

1 participant