Skip to content

Change commonJS into ES6 import#1

Open
2donny wants to merge 7 commits intomainfrom
master
Open

Change commonJS into ES6 import#1
2donny wants to merge 7 commits intomainfrom
master

Conversation

@2donny
Copy link
Copy Markdown

@2donny 2donny commented May 25, 2021

Code review

크게 3가지 정도 고쳐봤습니다. 자유롭게 피드백 주세요 :)

  1. commonJS 문법인 require를 ES6 import 문법으로 바꿨습니다. (자동 완성, name alias 등 많은 이점 제공)
  2. Index.js 파일을 로직에 맞게 init, server 파일로 쪼갰습니다.
  3. require('dotenv').config는 최초에 init.js에서 실행하므로 다른 비즈니스 로직 파일에서는 지웠습니다.
  4. server.js 파일에서 라우터 middleware를 사용하여 라우터를 분리하고, 반복되는 prefix url을 제거했습니다.

@2donny 2donny changed the title divide into init, server, db Change commonJS into ES6 import May 25, 2021
@2donny 2donny requested a review from diydriller May 25, 2021 09:40
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.

1 participant