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.

Sign up for GitHub

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

Jump to bottom

be eslint rule 변경 #85

Open
mosball opened this issue Nov 12, 2019 · 0 comments
Open

be eslint rule 변경 #85

mosball opened this issue Nov 12, 2019 · 0 comments
Assignees
@younguna @OriginJang @mosball @JeongHyeonJo
Labels
be Back-end related eslint setup

Comments

@mosball
Copy link
Contributor

mosball commented Nov 12, 2019 •
edited by JeongHyeonJo
Loading

Project와 week과 상관없이 계속해서 사용되고 reference되는 이슈

  1. linebreak-style 비활성화
  2. no-restricted-globals 비활성화
  3. class-methods-use-this 비활성화
  4. no-plusplus 비활성화
  5. no-param-reassign 비활성화
mosball added a commit that referenced this issue Nov 12, 2019
@mosball
Setup : no-restricted-globals eslint rule 변경
6071ccd 
상세 : -
이유 : airbnb eslint rule에서 글로벌 함수사용을 제한하여 no-restricted-globals rule 비활성화

#85
@younguna younguna reopened this Nov 14, 2019
@OriginJang OriginJang added the be Back-end related label Nov 18, 2019
mosball added a commit that referenced this issue Nov 19, 2019
@mosball
Setup : eslint rule 변경
1673f31 
이유 : class내부의 함수에서 this 키워드를
쓰지 않을 수도 있기 때문에 class-methods-use-this
rule 비활성화

#85
younguna added a commit that referenced this issue Nov 19, 2019
@younguna
Setup : eslint룰 no-plusplus를 해제함
ece80f5 
상세: i++를 가능하게 변경

#85
younguna added a commit that referenced this issue Nov 25, 2019
@younguna
Setup : eslint 설정 변경
06d082b 
상세:
- client/server workspace에서 eslint no-undef 해제
이유:
- jest는 글로벌로 작동하기 때문에 따로 임포트하여 사용하지 않음

#127 #85
@JeongHyeonJo JeongHyeonJo reopened this Nov 26, 2019
younguna added a commit that referenced this issue Nov 26, 2019
@younguna
Setup : server eslint no-param-reassign 설정해제
58ebe0f 
상세:
- no-param-reassign 서버에서 설정해제
- socket 자체에 키값 할당 할 수 있기 위해 해제
 #85
@younguna younguna changed the title eslint rule 변경 be eslint rule 변경 Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@younguna younguna

@OriginJang OriginJang

@mosball mosball

@JeongHyeonJo JeongHyeonJo

Labels
be Back-end related eslint setup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@younguna @OriginJang @mosball @JeongHyeonJo