-
Notifications
You must be signed in to change notification settings - Fork 42
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
ci: use semantic-release for releases #829
Conversation
97c6069
to
4d6c6e2
Compare
4d6c6e2
to
919f383
Compare
516c1bf
to
2cacc54
Compare
@@ -151,6 +151,11 @@ The release process is also automated in the way any specific commit from the ma | |||
1. Click the green `Run workflow` button to trigger the release workflow. | |||
1. Then you can go to the `https://www.npmjs.com/package/@elastic/synthetics` to validate that the bundles have been published. | |||
|
|||
#### Versioning |
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.
How does it pick the existing versioning if we set it to 0-0-0
in the package JSON, are we doing it based on existing tags?
Also can you post a dry-run output? I am not able to run it localy.
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.
Yes it's based on existing tags.
Will post the log output in a couple of minutes. I will also add a how to test section in the description.
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 justed added the "How to test section". The log output can be found in the collapsible details.
package-lock.json
Outdated
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.
There is something wrong with this file, Could you wipe it off and generate a new one by using the same node version?
nvm use && npm i
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.
.releaserc
Outdated
[ | ||
"@semantic-release/npm", | ||
{ | ||
"npmPublish": false, |
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.
"npmPublish": false, | |
"npmPublish": false |
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.
6bd69f6
to
c8a3299
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.
LGTM 🎉
Details
This change utilises semantic-release for releasing.
semantic-release does not support providing OTP by default. This is the reason why we are disabling the
npmPublish
option in the @semantic-release/npm plugin. Instead, we are using the@semantic-release/exec
plugin to run the publish command on our own.How to test
log output