-
Notifications
You must be signed in to change notification settings - Fork 2
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
Refactor/#427 ts-loader를 fork-ts-checker-webpack-plugin으로 대체한다 #428
Conversation
- South Korea 기준 72.2% 지원 - 참고: https://browsersl.ist/#q=last+2+years®ion=KR
5485623
to
cf52f25
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생많았습니다 코난!!! ts 컴파일과 타입체킹을 병렬적으로 하는 것이 신기하네요. 코난 캠퍼스 오면 한번 더 설명해주세요 ㅎㅎ 너무 궁금해요!
++ 추가 : last 2 years, not dead 어떠신가요?
not dead: 지원이 중단되지 않은 브라우저
frontend/babel.config.js
Outdated
// TODO: 브라우저 지원 범위 | ||
chrome: '51', | ||
}, | ||
targets: 'last 2 years', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 👍 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 not dead
도 추가하였습니다.
📝작업 내용
타입체킹 담당을 ts-loader에서 fork-ts-checker-webpack-plugin으로 대체하였습니다.
기존에 ts-loader가 타입스크립트 컴파일을 관여하였다면,
현재는 babel-loader의
@babel/preset-typescript
이 타입스크립트 컴파일을 담당하고 있습니다.ts-loader
가 있는데 왜@babel/preset-typescript
가 설치되어 있었나?기존
ts-loader
와 다른 부분은 타입체킹이 병렬적인 프로세스로 돌아간다는 점입니다.이 플러그인은 모듈을 해석할 때 웹팩이 아니라 타입스크립트의 모듈 해석을 사용한다는 것이 중요합니다.
💬리뷰 참고사항
개선전과 개선후 각각 10번씩 빌드하여 평균을 내었을때,
개선전 평균 6478.9ms, 개선후 평균 3656ms입니다.
소소하지만 쌓이고 쌓이면 클 것이라고 생각합니다 😊
추가된 설정 설명
tsconfig.json
의"jsx": "react-jsx",
속성이 jsx 변환을 담당해주었습니다.이제는 babel-loader를 통해 컴파일하기 때문에
@babel/preset-react
가 해당 부분을 담당하면서runtime: 'automatic'
속성이 추가되었습니다. (모든 파일에import React from 'react';
가 적용되도록_jsx
문법으로 컴파일하는 것입니다.)diagnosticOptions에 syntactic을 true로 바꾸어 문법 검사를 실시하도록 하였습니다.
issue - 타입체킹에 포함할 파일과 제외할 파일을 설정하였습니다.
#️⃣연관된 이슈
close #427