-
Notifications
You must be signed in to change notification settings - Fork 2
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
maintenance release workflow #93
Conversation
steps: | ||
- uses: actions/checkout@v3 | ||
with: | ||
fetch-depth: 0 | ||
- uses: actions/setup-node@v2-beta | ||
- uses: actions/setup-node@v3 |
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.
バージョン上げ
@@ -15,39 +15,37 @@ jobs: | |||
releasing: | |||
name: releasing | |||
runs-on: ubuntu-latest | |||
timeout-minutes: 10 |
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.
タイムアウト追加
node-version: 18 | ||
cache: npm | ||
registry-url: 'https://registry.npmjs.org' |
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.
- バージョン上げ
- キャッシュを使う
.npmrc
の設定をsetup-nodeにさせる
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.
nodeバージョン上げてますしリリースフロー確認がてらリリースしますか・・:eyes:
git config --global user.name 'github-actions[bot]' | ||
git config --global user.email '41898282+github-actions[bot]@users.noreply.github.com' |
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.
GitHub Actions のアカウントとして設定する(と、見た目がカッコイイ)
gh pr create | ||
--assignee | ||
--base "${{ github.event.repository.default_branch }}" | ||
--body '' | ||
--head "${{ github.ref_name }}" | ||
--title "chore(release): publish" |
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.
curlで頑張ってPOSTするよりかは現代ではghがあるのでそれを使う
https://github.com/peter-evans/create-pull-request を使うのでもよかったかもしれない
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.
だいぶ変わりましたね・・:tada:
permissions: | ||
contents: write | ||
id-token: write | ||
pull-requests: write |
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.
#93 (comment) により追加しました。
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.
release のワークフローも古かったので直しました。release のワークフローは頻繁に試せるわけでないので修正後にきちんと動作するのかは少し不安ではあるのですが……