Skip to content

Leave pull request numbers out of patch notes - #36

Merged
Drefvelin merged 1 commit into
mainfrom
fix/hide-patchnote-pr-numbers
Sep 25, 2026
Merged

Drefvelin merged 1 commit into
mainfrom
fix/hide-patchnote-pr-numbers

Conversation

@Drefvelin

@Drefvelin Drefvelin commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • GitHub squash titles like (#33) are removed before a patch note is stored.
  • Notes already saved still hide that number when the week is shown to staff or players.

Test plan

  • python3 -m unittest backend/src/patchnotes/test_summarize.py backend/src/api/test_patchnotes_routes.py
  • A line stored as Added evil RP sessions (#33) is returned as Added evil RP sessions.

Made with Cursor

Summary by CodeRabbit

  • Improvements
    • Pull request numbers in parentheses are now removed from patch note bullets, keeping published notes focused on the change itself.

GitHub adds (#33) to a squash title. That number is not something players need to see, so it is removed from new notes and from notes already stored.

Co-authored-by: Cursor <[email protected]>
@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 04598ac5-bc7a-4e57-b974-8f0c3114a71d

📥 Commits

Reviewing files that changed from the base of the PR and between 9464947 and 50335cd.

📒 Files selected for processing (4)
  • backend/src/api/patchnotes_routes.py
  • backend/src/api/test_patchnotes_routes.py
  • backend/src/patchnotes/summarize.py
  • backend/src/patchnotes/test_summarize.py
 ________________________________________________________________
< Fully armed and operationally intelligent code reviewer bunny. >
 ----------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@Drefvelin
Drefvelin merged commit ff402f3 into main Sep 25, 2026
1 of 2 checks passed
@Drefvelin
Drefvelin deleted the fix/hide-patchnote-pr-numbers branch September 25, 2026 11:11
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