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

2024-03-14 - npm link ってサブディレクトリに貼ると循環参照になるよね・・・? #279

Closed
kachick opened this issue Mar 13, 2024 · 2 comments
Labels
blog help wanted Extra attention is needed

Comments

@kachick
Copy link
Owner

kachick commented Mar 13, 2024

最近 octokit 周りのパッケージが軒並みESMに上がったんだけどそこまでテストが充実してないからかこぞって壊れてしまっている。なんとか動くように手元では変更できても、なぜそれで直るのかを説明できないからprとか上げにくい
どれもこれもnpmの複雑怪奇なパッケージ化を通り抜けた後にあれが漏れてるこれがおかしいみたいな壊れかたしてるから普通のunit test 足してどうにかなるもんじゃないよなー、みんなどうしてるんだと思ったら npm link というサブコマンドがあるらしい。しかし問題のリポジトリはテストコードを直接のサブディレクトリに置いてるから、このまま貼ったら循環参照にならんか・・・?なんか方法あるんかと調べたら8年前に無視されて閉じられてた。諸々辛い

npm/npm#11545

@kachick kachick added help wanted Extra attention is needed blog labels Mar 13, 2024
@kachick kachick closed this as completed Mar 14, 2024
@kachick
Copy link
Owner Author

kachick commented Apr 5, 2024

npmの複雑怪奇なパッケージ化

これは自分の勉強不足なだけで別に普通なのかもしんない。ようは知らないだけな可能性は高い

そこまでテストが充実してないからか

レイヤが足りないという話ではあるけど軽視しているわけではなさそうなので言い過ぎ

なんとか動くように手元では変更できても、なぜそれで直るのかを説明できないからprとか上げにくい

存在は知ってたのにここが使い所と思い出せなかったのを思い出した。 こと TypeScript に関しては https://github.com/arethetypeswrong/arethetypeswrong.github.io がぴったりだった。

octokit/graphql-schema@a347201 => テスト追加のPRは今投げてる(ここの 925 なんだけど、こういう時にやっぱ向こう側にリンクが見えちゃうんだよなー、ノイズすぎでしょやだなーと思うとうっとなって貼りたいリンク貼れなかったりするのが times 運用の微妙なところ。)

@kachick
Copy link
Owner Author

kachick commented Apr 5, 2024

存在は知ってたのにここが使い所と思い出せなかった

これは README を読んでブラウザからなんか1ライブラリぐらい叩いたけど、その瞬間に自分が抱えている問題が無くてそれ以上追わなかったというか具体的な用途が思い浮かばなかったからだと思う
今思うとそれより前に出くわしてた vitejs/vite#8701 と格闘してる時もしあれば有益だったかもしれない。(し、そうでもなかったかもしれない。ぐらいに理解してないけど、それすら思い浮かばないというのはまずい)

というところに思い至らなかったのはつまり https://scrapbox.io/nishio/%E8%AA%8D%E7%9F%A5%E3%81%AE%E8%A7%A3%E5%83%8F%E5%BA%A6 とか https://scrapbox.io/nishio/%E4%BC%BC%E3%81%9F%E3%82%82%E3%81%AE%E3%81%8C%E9%81%8E%E5%8E%BB%E3%81%AB%E3%81%82%E3%81%A3%E3%81%9F%E6%B0%97%E3%81%8C%E3%81%99%E3%82%8B でいうところの認知の解像度が低い状態だったからやな

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant