-
Notifications
You must be signed in to change notification settings - Fork 3
feat(scripts): adds scripts for managing releases #81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
👋 MStreet3, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
4f4cf6a to
5ffe60f
Compare
| Once the release is ready to be tagged execute the publish flow: | ||
|
|
||
| ```bash | ||
| $ make publish_release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's unlikely we'll always keep the capabilities in sync with the base SDK.
| - capabilities/scheduler/cron | ||
| - capabilities/networking/http | ||
| - capabilities/blockchain/evm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we discover capabilities by walking the folders?
| # Create a new branch from the release branch | ||
| describe "Creating fix branch..." | ||
| git checkout -B $fix_branch origin/$release_branch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this what we want? It seems wrong to make releases from the non-main branch.
| fi | ||
|
|
||
| # Ask for the stage of release | ||
| read -p "Version Stage [beta/stable]: " versionStage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add alpha and rc as optional, maybe confirm with @wentzeld and @ejacquier what tags we potentially want.
| "capabilities/scheduler/cron/" | ||
| "capabilities/networking/http/" | ||
| "capabilities/blockchain/evm/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use find to get them so we can't miss new ones. Ideally you specify which one you want and we have a helper to do it for all of them. The odds we update everything at once after GA are low.
5ffe60f to
7ce8192
Compare
No description provided.