Skip to content

Conversation

@simonbaird
Copy link
Member

I used this script to create three PRs for EC-1343 just now.

@simonbaird simonbaird force-pushed the add-ubi-bump-script branch 2 times, most recently from e4d02a5 to ae1014b Compare June 30, 2025 15:06
# Push the branch ready to make a PR
git push origin $PR_BRANCH:$PR_BRANCH
# Todo maybe: gh pr create ...
echo "Be careful to choose the right target branch!"
Copy link
Contributor

Choose a reason for hiding this comment

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

This warning comes when everything is already done, i would say you can either remove it, or move up as a confirmation, something like: "You are bumping branch xyz, confirm?"

@st3penta
Copy link
Contributor

st3penta commented Jul 1, 2025

Nice!
Consider adding an help/readme to easily know what are the available args

@simonbaird simonbaird force-pushed the add-ubi-bump-script branch 2 times, most recently from e2810b9 to 80a4205 Compare July 1, 2025 15:00
@simonbaird
Copy link
Member Author

I updated the instructions on how to create the PR to make it more obvious.

Nice! Consider adding an help/readme to easily know what are the available args

I think the args are for hacking/debugging purposes only so I feel lazy about document them. 😅

@simonbaird simonbaird force-pushed the add-ubi-bump-script branch from 80a4205 to a601b07 Compare July 1, 2025 15:05
@simonbaird
Copy link
Member Author

simonbaird commented Jul 1, 2025

Thanks for the review! I'll probably merge when/if it goes green (...to resist the urge to keep tweaking it.)

@simonbaird simonbaird merged commit 90ccd9b into conforma:main Jul 6, 2025
13 of 14 checks passed
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.

2 participants