Skip to content
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

Update lint-staged #86

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Update lint-staged #86

merged 1 commit into from
Dec 14, 2024

Conversation

massongit
Copy link
Contributor

lint-stagedをアップデートします。

@azu
Copy link
Member

azu commented Dec 14, 2024

ロックのアップデートとバージョンのアップデートの区別が難しくてレビューの認知負荷が高いです。(通知がたくさんきています)
dependenciesのアップデートはユーザー影響ありますが、devDepsはないのと、lockもないので、これをdiff見る前に区別したいです。
depsとdevDepsの区別は欲しいのと、Lockはあまり有意義ではないので自動化するなり別の方法を取りたいです(renovatebotとかdependabotを特定のリポジトリで絞ってるのは、通知だらけになってしまう問題があるからです。基本的にはdependabotのlock updateはautomergeしているつもりですが、リポジトリが分かれているのでちゃんと管理ができてないです)

どういう方針でアップデートしていきたいのかを一度どこか(Discussionとかに?)にProposalをまとめてやっていただきたいです。今週は忙しいのでそろそろ対応できなくなりそうです。

@azu azu added the Type: Dependencies Dependency issues or Changes to dependency files label Dec 14, 2024
@azu azu merged commit 7d34049 into textlint-ja:master Dec 14, 2024
2 checks passed
@massongit massongit deleted the update_packages_2 branch December 14, 2024 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Dependencies Dependency issues or Changes to dependency files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants