-
Notifications
You must be signed in to change notification settings - Fork 351
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
docs(main): auto release node using just #2537
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2537 +/- ##
=======================================
Coverage 65.86% 65.86%
=======================================
Files 133 133
Lines 16795 16795
=======================================
Hits 11062 11062
Misses 5733 5733 |
0160474
to
f9be60d
Compare
8e0c924
to
ce3432e
Compare
sudo env PATH=$PATH just version-up ${{ inputs.version }} | ||
- uses: peter-evans/create-pull-request@v5 | ||
with: | ||
title: 'docs: Automated docs Update for ${{ inputs.version }}' |
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.
Not only docs but also Cargo.toml.
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.
Why don't you use tagpr?
https://github.com/Songmu/tagpr
Looks like this would be a good approach. |
5bc82d5
to
5b2546b
Compare
be4d74e
to
32c610a
Compare
|
version: | ||
description: 'Version to release' | ||
required: true | ||
default: '0.3.0' |
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 is not necessary.
default: '0.3.0' |
@@ -0,0 +1,7 @@ | |||
[tagpr] |
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.
Please add a comment about how we don't need to edit it.
justfile
Outdated
if [[ "{{version}}" == "0.3.0" ]]; then | ||
return | ||
fi |
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.
Why do we need it?
if [[ "{{version}}" == "0.3.0" ]]; then | |
return | |
fi |
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.
The main thing is that when this PR is submitted, a current version of the PR will be automatically generated. Another way is to change the current default version number to 0.3.1
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 a good time to release 0.3.1. I'd like to release the new version using the new workflow you created! So can we delete it?
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.
Of course, I've deleted it. When the current PR is merged, it will automatically create a new PR. When the new PR is merged, it can be automatically released.
.github/workflows/tagpr.yaml
Outdated
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
- name: Trigger Release Workflow(only when tagged) | ||
uses: actions/github-script@v6 | ||
if: "steps.tagpr.outputs.tag != '' && steps.tagpr.outputs.tag != 'v0.3.0'" |
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.
Please check format instead of v0.3.0.
inputs: | ||
version: | ||
description: 'Version to release' | ||
required: true | ||
default: '0.3.0' |
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 it difficult to choose major
, minor
, or patch
to take place of a version?
d770988
to
073d661
Compare
@cuisongliu Sorry for the late. May I ask you to rebase this PR from the main branch? We extended the timeout for CI. |
4fb2c65
to
7584b59
Compare
done |
d53ac07
to
fd0a822
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.
Looks good except for the conlict
Signed-off-by: cuisongliu <[email protected]>
fd0a822
to
c8d2228
Compare
@cuisongliu Thanks! |
No description provided.