Skip to content
This repository was archived by the owner on Oct 24, 2019. It is now read-only.

Conversation

@dongwoo-kim
Copy link

@dongwoo-kim dongwoo-kim commented Mar 25, 2019

Description

  • Added prettier config / Applied prettier format
  • Added type definition for instance methods
  • Added vue as a peer dependency

@dongwoo-kim dongwoo-kim requested a review from sohee-lee7 March 25, 2019 05:01
{
"name": "@toast-ui/vue-grid",
"version": "1.0.4",
"version": "1.0.5",
Copy link
Contributor

Choose a reason for hiding this comment

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

package-lock.js의 version도 함께 바껴야할 것 같습니다.

Copy link
Author

Choose a reason for hiding this comment

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

이거 반영하긴 했는데..
생각해보니 package-lock.json 버전은 매번 업데이트 하면 안되겠네요.
이 파일은 직접 관리하는 파일이 아니라 자동 생성되는 파일이라서 수동으로 수정하면 안될 것 같아요.
이 패키지의 버전이 올라가도 디펜던시가 추가되거나 업데이트 된 게 아니라면 그대로 유지하는 게 맞는 것 같습니다.

@sohee-lee7
Copy link
Contributor

리뷰 완료입니다.!!! Typescript 어렵네요..FunctionKeys를 이해하기까지 오래거렸네요 ㅜㅜ FunctionKeys는 다른 vue wrapper 에서도 ts적용할때 사용할 수 있을 것 같아서 좋네요! 고생하셨습니다 👍 💯 🥇

@dongwoo-kim dongwoo-kim merged commit 1121ae5 into master Mar 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants