fsubal
fsubal
> ディレクトリの中のファイルを取る処理が1段分しかなく、svg/16/hoge.svgのようなファイルの差分を検出できていなかった この説明だと最初からずっと動かなかったはずなので、ある時から動かなくなった理由としては違和感あるな…（違う理由な気がする
手動実行中 https://github.com/pixiv/charcoal/runs/7849114017?check_suite_focus=true → 0 files changed だった。動作確認ってどうやりました？
@toshusai > `OverlayProps`のプロパティがStorybookのcontrolsに出ていないのだけ気になりました。 他は問題なさそうです👍 `isOpen` とかは今だとボタンで制御してますが、controls にあった方が良いですかね。 これ以外だと `isDismissable` とかは変更できると良いのかな（どのへんがいじれるとうれしいかのイメージが欲しいかも）
Storybook の argTypes が自動で推論される仕組みが上手く行ってなくて、 `isDismissable` とかが controls に生えてこないのが気になりポイントだったっぽい。 react-docgen の挙動とか探ったがなぜうまく行かないのか分からなかった。 直す優先度はあまり高くないので最悪このままでマージする
ちなみに私は pnpm 推しです
peer dependencies が pnpm の機能で auto install できるのを良しとすべきなのかが分かってない（ 依存の依存の peer dependencies を我々側は明示すべきなのか？ 利用側のプロダクトが peer dependencies の入れ忘れをするなら意味がないのか？ ）。
eslint-plugin-import と pnpm は別に両方入れても良い（ いや、前者だけで済むよみたいな根拠がない限りは ）
1個懸念を書くのを忘れてた。pnpm は dependabot でサポートされていなそう https://github.com/dependabot/dependabot-core/issues/1736 yarn v2 もつい最近までなかったのでいまさら感はあるが、やはり renovate しか選択肢がないという状況になりそう
pnpm サポートされてた https://github.com/dependabot/dependabot-core/issues/1736#issuecomment-1588051751
- react-aria 側で OverlayProps の定義が変わったかもしれないので確認 - ModalPlumbing も index.ts から export したほうが良い