Skip to content

Conversation

@OnestarLee
Copy link
Contributor

modules에서 고객사에게 영향이 없을것 같다고 예상되는 에러들 위주로 수정 하였습니다.

@OnestarLee OnestarLee requested review from AhyoungRyu, HoonBaek, bang9, chohongm and chrisallo and removed request for HoonBaek and bang9 April 5, 2024 07:28
@OnestarLee OnestarLee self-assigned this Apr 5, 2024
@chohongm
Copy link
Contributor

chohongm commented Apr 8, 2024

can you run yarn lint:fix

Copy link
Collaborator

@chrisallo chrisallo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! 커멘트 드린 부분만 수정해주시면 될것같습니다!


useLayoutEffect(() => {
const timeouts = [];
const timeouts: NodeJS.Timeout[] = [];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ReturnType<typeof setTimeout>[]로 수정하면 좋을것 같습니닷!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 하였습니다

const [requestString, setRequestString] = useState('');
const [debouncingTimer, setDebouncingTimer] = useState(null);
const [requestString, setRequestString] = useState<string>('');
const [debouncingTimer, setDebouncingTimer] = useState<NodeJS.Timeout | null>(null);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ReturnType<typeof setTimeout> | null

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 하였습니다.

@OnestarLee OnestarLee force-pushed the feat/v3/strict-typescript/CLNP-2785-strict-typescript4 branch from 699d5f1 to 7e4f6e4 Compare April 9, 2024 05:33
@OnestarLee
Copy link
Contributor Author

run yarn lint:fix 돌려서 확인 하였습니다

@OnestarLee OnestarLee merged commit 45d5413 into feat/v3/CLNP-2496-strict-typescript Apr 9, 2024
@OnestarLee OnestarLee deleted the feat/v3/strict-typescript/CLNP-2785-strict-typescript4 branch April 9, 2024 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants