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

refactor: GitHub Actions 워크플로우 최적화 완료 #93

Open
wants to merge 84 commits into
base: develop
Choose a base branch
from

Conversation

limehee
Copy link
Collaborator

@limehee limehee commented Dec 4, 2024

Summary

#92

CI 워크플로우의 테스트 및 빌드 효율성을 높이고 유지보수성을 개선합니다.
테스트 프로파일을 Git에 업로드하여 관리하고, 공통 설정을 재사용 가능하도록 변경하였으며, 빌드 단계에서 JDK 대신 JRE 이미지를 사용하여 리소스를 최적화했습니다.

Tasks

  • 테스트 프로파일 관리

    • application-test.yml 파일을 Git에 업로드하고, CI 워크플로우에서 이를 직접 로드하도록 설정.
  • 워크플로우 간결화

    • Gradle Cache 설정 및 Set up JDK 등의 공통 작업을 setup-common.yml로 분리하여 중복 제거.
    • Check build status 단계를 제거하여 불필요한 작업을 생략.
  • Build 작업 최적화

    • 빌드 단계에서 JDK 대신 JRE 이미지를 사용하도록 변경하여 리소스 사용량 최소화.

ETC

  • CD 파이프라인에서 사용되는 프로파일을 단일 파일로 통합하였습니다.

@limehee limehee added 🌎 Intra 서버 인프라 및 도메인 관련 🔨 Refactor 코드 수정 및 개선 labels Dec 4, 2024
@limehee limehee requested a review from Copilot December 4, 2024 08:17
@limehee limehee self-assigned this Dec 4, 2024
@limehee limehee linked an issue Dec 4, 2024 that may be closed by this pull request

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 6 out of 11 changed files in this pull request and generated no suggestions.

Files not reviewed (5)
  • .dockerignore: Language not supported
  • jenkins/Jenkinsfile: Language not supported
  • stempo-api/build.gradle.kts: Language not supported
  • stempo-application/src/test/resources/application-test.yml: Language not supported
  • .github/workflows/spring-boot-gradle-ci.yml: Evaluated as low risk
Copy link

codecov bot commented Dec 12, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

Copy link

🛠️ Test Summary (success)

📄 View Detailed Test Logs

Metrics Test Result Details
Total Tests 482
Tests Passed 482
Tests Failed 0
⚠️ Tests Skipped 0
⏱️ Execution Time 75.92s

Copy link

sonarcloud bot commented Dec 12, 2024

Copy link

📉 JaCoCo Coverage Verification Results

Class Metric Actual Threshold Type Expected
com.stempo.ApiApplication lines covered 0.00 minimum 0.50
com.stempo.annotation.SuccessApiResponseCustomizer branches covered 0.66 minimum 0.70
com.stempo.config.WebConfig.1 branches covered 0.00 minimum 0.70
com.stempo.config.WebConfig.1 lines covered 0.20 minimum 0.50
com.stempo.support.yaml.YamlEnvironmentPostProcessor branches covered 0.30 minimum 0.70

(If empty, there are no coverage violations.)

HTML Coverage Report

You can download the HTML coverage report from the Artifacts of this run:
View Coverage HTML Artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌎 Intra 서버 인프라 및 도메인 관련 🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Actions 워크플로우 최적화
1 participant