remark-lint-local-links-valid 모노레포화 #434
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
CirnoV
force-pushed
the
feat/lint-monorepo
branch
2 times, most recently
from
d827490 to
070df55
Compare
package.json
Outdated
| @@ -37,6 +38,7 @@ | |||
| "@iconify-json/ic": "^1.1.17", | |||
| "@iconify-json/material-symbols": "^1.1.78", | |||
| "@iconify-json/simple-icons": "^1.1.100", | |||
| "@portone-io/remark-lint-local-links-valid": "^0.1.1", | |||
There was a problem hiding this comment.
앗 이거 근데 생각해보니까 린트 룰 패키지를 개발자센터 레포 안에 둘 거면 굳이 깃헙 패키지에 안 올리고 workspace:로 가져다 써도 되지 않을까요? (원래 질문 답변 드릴 때는 remark 룰 관리하는 모노레포를 별도로 파신다고 이해했었다는)
그리고 깃헙 패키지를 쓰면 portone-io 접근권한 없는 사람들은 패키지 접근이 안 될 거라서 외부기여 받기도 불편할 것 같아요 😢
There was a problem hiding this comment.
visibility를 public으로 설정할 수 있어서 그렇게 하고 시크릿 모드에서도 접근이 가능한걸 확인했었는데, Github Packages 문서를 보니까 npm registry로써 접근하기 위해서는 public/private 관계없이 Token 인증이 필요한 것 같더라구요. 그래서 권한 없이도 로그인만 하면 패키지를 사용할 수는 있을거에요.
workspace:로 사용하면 로컬에서 수정하기가 편리하겠지만, slack에서 공유했던 대로 패키지를 언제 transpiling 할지 정해야 할 것 같아요.
There was a problem hiding this comment.
https://tsx.is/node#hooks 이거 써서 Remark에서 온디맨드로 TS 트랜스파일링할 수 있도록 하면 될 것 같아요 (VSCode 익스텐션이 잘 돌 수 있을지는 또 봐봐야 하겠지만...)
| @@ -0,0 +1,2 @@ | |||
| packages: | |||
| - 'packages/*' | |||
There was a problem hiding this comment.
이러면 아마 루트 패키지(developers.portone.io)가 워크스페이스에 안 들어갈 거에요
- '.' 넣어주셔야 할 듯해요
There was a problem hiding this comment.
이거 넣고 안넣고가 어떤 차이가 있는지 잘 모르겠어요. 기존 명령어도 잘 작동되는것처럼 보이고, 패키지 설치할 때 워크스페이스 루트에서는 pnpm (-w)--workspace-root install <package@version> 처럼 똑같이 옵션 줘야하는 것 같아서요
There was a problem hiding this comment.
오잉 이거 안 넣고도 루트 패키지에서 모노레포 내의 패키지를 참조한다거나 하는 등의 동작이 모두 잘 작동하나요? 👀
There was a problem hiding this comment.
네 지금 상태에서도 루트 패키지에서 설치 잘 되고 있어요
There was a problem hiding this comment.
오.... 뭔가 루트 패키지는 기본적으로 추가시켜 준다거나 하는 동작이 있나 보네요
CirnoV
force-pushed
the
feat/lint-monorepo
branch
2 times, most recently
from
4eaddad to
111a2fc
Compare
| @@ -142,14 +143,15 @@ | |||
| "satori": "^0.10.13", | |||
| "sharp": "^0.33.3", | |||
| "string-width": "^5.1.2", | |||
| "ts-node": "^10.9.2", | |||
There was a problem hiding this comment.
tsx를 사용했을 때 여전히 import 오류가 발생해서 ts-node로 구현했습니다
CirnoV
force-pushed
the
feat/lint-monorepo
branch
from
111a2fc to
8dcceaf
Compare
CirnoV
force-pushed
the
feat/lint-monorepo
branch
from
fa084de to
e991e16
Compare
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
@portone-io/remark-lint-local-links 에 패키지 publishpackage.json의 버전에 따라 자동으로 publish 하는 workflow 작성README에pnpm login가이드 추가Vercel 환경변수 설정에NPM_RC추가workspace:^참조로 패키지 종속성 추가ts-node를 이용해서 런타임 transpiling 구현Vercel 환경변수 설정에 사용한 토큰은 일단 개인 발급한 무기한 토큰으로 설정해뒀는데 괜찮을지 모르겠네요.. 🤔PR 반영되면 기존 저장소는 여기로 issue 옮기고 나서 삭제할 예정입니다.