Skip to content

[codex] Remove stale changelog PR template instruction#3984

Merged
szdziedzic merged 1 commit into
mainfrom
szdziedzic-codex/remove-stale-changelog-template-instruction
Jul 7, 2026
Merged

[codex] Remove stale changelog PR template instruction#3984
szdziedzic merged 1 commit into
mainfrom
szdziedzic-codex/remove-stale-changelog-template-instruction

Conversation

@szdziedzic

@szdziedzic szdziedzic commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Why

The PR template tells contributors to add changelog entries by commenting with /changelog-entry ..., but the repository changelog workflow only checks whether CHANGELOG.md changed on pull request events. There is no issue_comment workflow handling that slash command, so the instruction points contributors at a no-op path.

#3982 (comment)

How

Removed the stale /changelog-entry instruction from .github/PULL_REQUEST_TEMPLATE. The rest of the template remains intact, including the guidance for using the no changelog label when a changelog entry is not required.

Test Plan

  • git diff --check
  • ./node_modules/.bin/oxfmt .

@szdziedzic szdziedzic added the no changelog PR that doesn't require a changelog entry label Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

@szdziedzic szdziedzic marked this pull request as ready for review July 7, 2026 12:36
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

Subscribed to pull request

File Patterns Mentions
**/* @douglowder

Generated by CodeMention

Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead.

@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.34%. Comparing base (081e1a9) to head (6b41d41).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3984      +/-   ##
==========================================
+ Coverage   59.33%   59.34%   +0.02%     
==========================================
  Files         936      936              
  Lines       41172    41186      +14     
  Branches     8671     8675       +4     
==========================================
+ Hits        24424    24438      +14     
  Misses      16654    16654              
  Partials       94       94              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@szdziedzic szdziedzic merged commit 502f713 into main Jul 7, 2026
11 of 12 checks passed
@szdziedzic szdziedzic deleted the szdziedzic-codex/remove-stale-changelog-template-instruction branch July 7, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants