-
Notifications
You must be signed in to change notification settings - Fork 10
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
dependabotでchangesetの自動生成をやめる #1346
Conversation
🦋 Changeset detectedLatest commit: 8c50c16 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
👷 Deploy Preview for ingred-ui processing.
|
1 similar comment
👷 Deploy Preview for ingred-ui processing.
|
✅ Deploy Preview for ingred-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
- name: check changesets | ||
run: make -f ci.mk check_changesets | ||
if: github.actor != 'dependabot[bot]' |
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.
dependabotはchangesetをしていないくもいいので除外
@@ -95,6 +19,7 @@ jobs: | |||
run: make -f ci.mk install | |||
- name: check changesets | |||
run: make -f ci.mk check_changesets | |||
if: github.actor != 'dependabot[bot]' |
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.
人間はチェックする、良さそう!
@@ -4,84 +4,8 @@ on: | |||
pull_request: | |||
|
|||
jobs: | |||
generate-changeset: |
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.
直でスクリプトを書いている場所を削れたの大変素敵でいい
背景
ref: https://cartaholdings.slack.com/archives/C02Q8F3FQBC/p1690246043246829
depenadabotの場合は、自動でchangesetを実行することを試みていたが、以下の課題があった。
package.json
が更新されない場合(yarn.lock
のみの更新)の対処が難しい、無理やりやったとてもyarn
に依存してしまうpatch
以外を上げたいような大きなバージョンアップもpatch
になるそのため、dependabotのPRはリリースノートには含めず、大きな変更があった場合のみ、depenadabot作成PRに手動でpushする方針とした。
やったこと