sirin05137 / CSE364_Project

2 stars 0 forks source link

Request for the code review #6

Closed yuujinleee closed 3 years ago

yuujinleee commented 3 years ago

코드 검토 해주세여

  1. dockerfile

    • (선택) 현재 MAVEN_HOME, M2_HOME 등의 환경변수는 설정해놓지 않음.
    • (line 26) WORKDIR 을 하는데 mkdir 지워도 되나
    • (line 33) ENTRYPOINT ["/bin/bash"] 대신 CMD ["/bin/bash"]?
  2. run.sh

    • 우리끼리 git clone을 OAuth로 해놓을 건지(현재) 아님 기본 주소로 다시 바꿔놓을 건지 정하면 됌. related to issue #1
    • 자바 커맨드는 일단 미 포함해놓은 상태 (제출시에는 포함해야함)
  3. pom.xml

yuujinleee commented 3 years ago

3. pom.xml 관련 -> issue #13