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

chore : CI/CD 구축 #7

Merged
merged 4 commits into from
Apr 24, 2024
Merged

chore : CI/CD 구축 #7

merged 4 commits into from
Apr 24, 2024

Conversation

tlarbals824
Copy link
Contributor

@tlarbals824 tlarbals824 commented Apr 23, 2024

🗃 Issue

🔥 Task

  • Dockerfile 추가
  • CI/CD 추가
  • 태그 자동 생성하도록 쉘스크립트 작성

📄 Reference

  • None

@tlarbals824 tlarbals824 added the 🌏 Deploy&Infra 인프라, 배포 관련 label Apr 23, 2024
@tlarbals824 tlarbals824 self-assigned this Apr 23, 2024
@tlarbals824 tlarbals824 requested a review from sominyun April 23, 2024 14:23
Copy link

제공된 코드는 Dockerfile이며 간단한 설정으로 Java 애플리케이션을 Docker 컨테이너 내에서 실행하기 위한 것입니다. 해당 코드는 Amazon Corretto 17 알파인 이미지를 기반으로 생성된 컨테이너 내에서 Asap-Backend.jar 파일을 실행하는 설정을 제공합니다.

여기에는 여러 개선사항이 있습니다:

  1. Method Naming:
    • Current State:
      • 코드 샘플은 Dockerfile이라는 파일명을 사용하며 명확하게 기능을 설명하지 않습니다.
    • Recommendation:
      • Dockerfile 대신 Dockerfile-for-Java 또는 Dockerfile-AsapBackend와 같이 목적을 나타내는 이름을 선택합니다.
FROM amazoncorretto:17-alpine
COPY /build/libs/Asap-Backend.jar /app.jar
EXPOSE 8080
ENTRYPOINT ["java", "-jar", "/app.jar"]

이러한 변경을 통해 코드를 더 명확하게 이해하고 다른 이용자들도 코드의 목적을 쉽게 파악할 수 있게 됩니다.

@tlarbals824 tlarbals824 marked this pull request as ready for review April 24, 2024 15:43
@tlarbals824 tlarbals824 merged commit 0dcd923 into develop Apr 24, 2024
1 check passed
@tlarbals824 tlarbals824 deleted the chore/#5-cicd branch April 24, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌏 Deploy&Infra 인프라, 배포 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant