Skip to content
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

itemModel 네이밍 변경 이슈 #31

Open
77unny opened this issue Mar 14, 2020 · 3 comments
Open

itemModel 네이밍 변경 이슈 #31

77unny opened this issue Mar 14, 2020 · 3 comments

Comments

@77unny
Copy link
Collaborator

77unny commented Mar 14, 2020

@reesekimm 내용 확인 부탁드려요!
변경 후 깨지는 부분 없는지 확인도 했습니다.

itemModel 을 vendingMachineModel 로 변경
변경 되면서 아래의 파일들이 수정 되었습니다.

  • main.js
  • controller.js
  • itemPanelView.js
  • statePanelView.js

삭제

  • itemModel.js

추가

  • vendingMachineModel.js
@77unny
Copy link
Collaborator Author

77unny commented Mar 14, 2020

흠.. 근대 파일 변경 및 수정인데.. 브랜치명이나......그냥 써도 되려나요..

@reesekimm
Copy link
Collaborator

reesekimm commented Mar 14, 2020

흠.. 근대 파일 변경 및 수정인데.. 브랜치명이나......그냥 써도 되려나요..

브랜치 rule을 만들면 어떨까요?
rule이라니까 좀 거창하게 들리는데 암튼 브랜치 관리할때
feature-xxx 외에 chore 같은 걸 만들어서 자잘한 네이밍 수정이나 스타일 변경 (e.g. ' ' -> " ") 등의 작업시 사용하면 좋을 것 같아요.

@77unny
Copy link
Collaborator Author

77unny commented Mar 14, 2020

!! 오옹 좋은거 같아요 ㅋㅋㅋ 그럼 다시 브랜치 만들어서 pr 날리겠심덩!

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

No branches or pull requests

2 participants