-
Notifications
You must be signed in to change notification settings - Fork 12
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: improve release workflow #767
Conversation
@@ -22,6 +22,7 @@ jobs: | |||
- uses: actions/setup-node@v3 | |||
with: | |||
node-version: 16 | |||
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にしてもらいます。
@@ -30,17 +31,14 @@ jobs: | |||
node_modules | |||
*/*/node_modules | |||
key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }}-v2 | |||
- run: npm install -g npm@9 |
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.
--provenance
で公開したいので npm9 にします。
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のように表示されます。
openameba/ameba-color-palette.css#93 (comment)
@@ -51,6 +49,8 @@ jobs: | |||
run: yarn lerna publish ${{ steps.extract_branch.outputs.version }} --conventional-commits --create-release github --yes --no-private | |||
env: | |||
GH_TOKEN: ${{ github.token }} | |||
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | |||
NPM_CONFIG_PROVENANCE: true |
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.
npm から直接公開するのではないため --provenance
の代わりに環境変数で対応します。
@@ -51,6 +49,8 @@ jobs: | |||
run: yarn lerna publish ${{ steps.extract_branch.outputs.version }} --conventional-commits --create-release github --yes --no-private | |||
env: | |||
GH_TOKEN: ${{ github.token }} | |||
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} |
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 にしているため NODE_AUTH_TOKEN
に NPM_TOKEN
を指定しています。
@@ -51,6 +49,8 @@ jobs: | |||
run: yarn lerna publish ${{ steps.extract_branch.outputs.version }} --conventional-commits --create-release github --yes --no-private | |||
env: | |||
GH_TOKEN: ${{ github.token }} | |||
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | |||
NPM_CONFIG_PROVENANCE: true | |||
- name: Create Pull Request | |||
run: > | |||
curl |
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.
gh
コマンドに変えようかと思いましたが、たしかAPI経由で実行しないとワークフローが実行されない関係でこの記述をしていた記憶があったのでとりあえず変更しませんでした。
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 workflow を改良しました。