BREAKING CHANGE(temen): Deprecated되어 browser-toolkit으로 이동한 함수들 제거 #117
Conversation
evan-moon
requested review from
Jogeonsang,
Junkim93,
pa-rang,
s-ong-c,
ssMinji,
theJunimo and
zi-gae
as code owners
| export function findQueryParamInClient(key: string) { | ||
| const queryString = location != null ? location.search : ''; | ||
| const query = parseQueryString(queryString); | ||
|
|
||
| return query[key]; | ||
| } |
There was a problem hiding this comment.
이 친구는 location 에 디펜던시가 있어서 browser-toolkit으로 옮겼읍니다
packages/temen/tsconfig.json
Outdated
| @@ -3,7 +3,7 @@ | |||
| "compilerOptions": { | |||
| "outDir": "./dist", | |||
| "declarationDir": "dist/types", | |||
| "lib": ["es2020"], | |||
| "lib": ["es2020", "dom"], | |||
There was a problem hiding this comment.
동욱님 브라우저 관심사가 있는 유틸들은 브라우저툴킷으로 이동한것 같은데,
temen에서 dom에 관한 타입정의 추가해주신 이유가 궁금합니다 😶😶
There was a problem hiding this comment.
temen에는 setTimeout을 사용하는 delay 함수와 window를 사용하는 isServer 상수 같은 녀석들도 있는데, 이 녀석들 때문에 필요하기는 합니다...!
근데 원래는 이 dom lib 설정이 루트에 있는 tsconfig.json에 포함되어 있기 때문에 잘 빌드되어야 정상인데, 갑자기 안 되네유....? browser-toolkit 같은 경우는 루트에 있는 dom 값을 잘 참조하고 있는데, temen이 갑자기 망가짐...
There was a problem hiding this comment.
동욱님 그런데 지금 lib쪽 옵션을 해당 temen이나 browser-toolkit 에서 덮어쓰고 있어서, 원래 루트에서 정의했던 옵션은 적용 안되지 않나요..!?
There was a problem hiding this comment.
microsoft/TypeScript#20110
요거 해당 이슈 링크입니다 ..!
There was a problem hiding this comment.
아하 이거 머지가 안 되는 옵션이라는 걸 놓쳤네유....ㅜㅜㅜ
9de77d0 에서 browser-toolkit과 temen의 tsconfig.json 내에서 lib 옵션을 제거했습니다.
체크해보기
export했나요?jsDoc포맷으로 잘 입력되어있나요?변경사항
제곧내입니다. Depreacted된 함수들을 제거하고 Major Update를 진행합니다.
집중적으로 리뷰 받고 싶은 부분이 있나요?
🙇