-
Notifications
You must be signed in to change notification settings - Fork 727
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
new(workflows/push): automate releases #1073
Conversation
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.
👋 Hi,
This is a reminder for maintainers to assign a proper release label to this Pull Request.
The bot will dismiss the review as soon as a valid release label has been assigned.
Thanks.
✅ Pull Request has a valid release label.
65e23e3
to
683e180
Compare
52e4620
to
b345221
Compare
// perform release | ||
try { | ||
const version = `${isPreRelease ? 'pre' : ''}${ | ||
isMajor ? 'major' : isMinor ? 'minor' : 'patch' |
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.
if only alpha-release
is set, this will default to patch
which I think is safe
run: | | ||
yarn build:sizes | ||
git config user.name github-actions | ||
git config user.email github-actions@github.com | ||
git add . | ||
git commit -m "build(${GITHUB_SHA}): auto-commit package sizes" | ||
git diff-index --quiet HEAD || git commit -m "build(${GITHUB_SHA}): auto-commit package sizes" |
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.
if there were no package size changes, this was failing before
scripts/performRelease/updateChangelog/mergeUpdateIntoChangelog.ts
Outdated
Show resolved
Hide resolved
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.
LGTM. Thanks for all the research. A lot of customization here.
🎉 This PR is included in version |
🏠 Internal
This PR adds automated releases to the github
push
workflow (i.e., merges intomaster
). I wrote most everything custom because I couldn't find anything compelling for both the automated release + changelog updates without also adopting conventional commits which I think would be tricky. Npm auth with lerna was tricky 🥴 .In the future the release cycle would work like the following:
patch-release
,minor-release
,major-release
,alpha-release
), nothing will happengithub-bot
posts on all PRs since the last release informing the author(s) which version the PR was released asI tested this as much as I could on a branch, so hopefully it will go smootly upon merge.
Example commits
Example changelog diff
Example posting on PRs
@kristw @hshoff