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
fix(release): publish correctly to github #2141
Conversation
330bb3a
to
4e7cd54
Compare
4e7cd54
to
bbc48ae
Compare
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.
small comment about git fetch but looks good otherwise
scripts/gitrelease.mjs
Outdated
@@ -14,10 +14,24 @@ if (tagExists.stdout !== '') { | |||
} | |||
|
|||
const releaseMessage = `chore(release): ${nextVersion} [skip ci]`; | |||
|
|||
echo`Checkout out branch`; | |||
await $`git fetch`; |
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.
we can probably speed things up by only fetching the branch we are interested in
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.
indeed!
## Describe your changes Follow up of #2141 Release worked but not the publish of the changelog - Fetch all commits by default - Improve grouping in changelog
Describe your changes
Turns out we don't have access to the full repo by default, have to fetch and checkout the proper branch and setup an author. You can check the commit it created on this branch 4ab23d2