Skip to content

Conversation

@wonjuneee
Copy link
Contributor

No description provided.

@wonjuneee wonjuneee self-assigned this Jul 4, 2025
@wonjuneee wonjuneee added the develop Gonna Merge on Develop branch label Jul 4, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR sets up local development workflows and CI configurations by adding test settings, Docker Compose definitions, and GitHub Actions workflows.

  • Add H2 in-memory datasource configuration for tests
  • Define Docker Compose services for Postgres, Redis, Spring Boot app, and Datadog agent
  • Introduce GitHub Actions workflows: auto-labeler and Java CI with Gradle
  • Provide branch-to-label mappings

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/test/resources/application-test.yml Add H2 in-memory database configuration for tests
compose.yaml Introduce Docker Compose for local services and Datadog agent integration
.github/workflows/labeler.yml Add workflow to automatically label issues and pull requests
.github/workflows/gradle.yml Add Java CI workflow using Gradle (build and dependency submission)
.github/labeler.yml Configure branch prefix patterns for automatic labeling
Comments suppressed due to low confidence (2)

compose.yaml:27

  • The volume mapping uses a single path without specifying a container path; you should specify both host and container paths (e.g., host_path:container_path) to ensure correct mount.
      - /var/app/log/eodiroo # 애플리케이션 로그를 호스트 볼륨에 마운트 (Datadog Agent가 읽을 수 있도록)

compose.yaml:28

  • Using depends_on with condition: service_healthy requires healthchecks on the dependent services; consider adding healthcheck definitions for Postgres and Redis to ensure they actually report healthy.
    depends_on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

develop Gonna Merge on Develop branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants