Skip to content

[WPB-23806] Opinionated script for creating wire-server PRs.#5178

Draft
fisx wants to merge 2 commits intodevelopfrom
WPB-23806-opinionated-script-for-creating-wire-server-prs
Draft

[WPB-23806] Opinionated script for creating wire-server PRs.#5178
fisx wants to merge 2 commits intodevelopfrom
WPB-23806-opinionated-script-for-creating-wire-server-prs

Conversation

@fisx
Copy link
Copy Markdown
Contributor

@fisx fisx commented Apr 11, 2026

https://wearezeta.atlassian.net/browse/WPB-23806

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Apr 11, 2026
Comment thread hack/bin/new-pr.sh

# Validate arguments
if [ $# -ne 3 ]; then
echo "Usage: $0 <wpb-number> <changelog-category> <description>"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm a bit worried that this will lead to very sparse PR descriptions. (Though, I've got no data to back this, of course.)

I'm wondering if it wouldn't be better to open $EDITOR here or have that as additional option. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants