search

dduk-ddak / coding-night-live

Web-based Communication Application for Codelabs
Other
62 stars 11 forks source link

naming convention #71

Open juice500ml opened 7 years ago

juice500ml commented 7 years ago

get/set --> remote/local camel / underscore / hypen

minhoryang commented 7 years ago

Flake8 please

punkyoon commented 7 years ago

09c0697af156e96b6173354cea9bfd673d929547

travis에서 확인할 수 있게 해두었어요.. 차근차근 고쳐볼게요!

juice500ml commented 7 years ago

https://git-scm.com/book/ko/v1/Git%EB%A7%9E%EC%B6%A4-Git-%ED%9B%85 travis 말고 git hook이 낫지 않을까싶습니다!

juice500ml commented 7 years ago

http://stackoverflow.com/questions/1837681/pre-commit-hook-for-jslint-in-mercurial-and-git 도 쓰죠!

punkyoon commented 7 years ago

flake8에서 잡혔던 문제는 다 해결했고, 변수 이름 / 메소드(or함수) 이름 / 클래스 이름 짓는 법만 맞추면 될 듯 합니다.

punkyoon commented 7 years ago

Python

Javascript

juice500ml commented 7 years ago

HTML이랑 CSS도 잡아야 합니다 ㅋㅋ

minhoryang commented 7 years ago

앜ㅋㅋㅋㅋ 이게 그거구나ㅋㅋㅋㅋ

punkyoon commented 7 years ago

@juice500ml 나머지는 광희가..정하는걸로..!

juice500ml commented 7 years ago

front쪽은 angluar/react 정해지면 그 스타일대로 가겠습니다~

punkyoon commented 7 years ago

https://github.com/dduk-ddak/coding-night-live/blob/master/manage_room/views.py#L16

제가 위에서 정한 기준에 안맞아요...★ 집에가서 수정할게요.