Skip to content

feat: #266 과거 그룹 진행일 옵션 수정#268

Merged
kelly6bf merged 1 commit into
devfrom
#-266
May 15, 2026
Merged

feat: #266 과거 그룹 진행일 옵션 수정#268
kelly6bf merged 1 commit into
devfrom
#-266

Conversation

@kelly6bf

@kelly6bf kelly6bf commented May 15, 2026

Copy link
Copy Markdown

😉 연관 이슈

Resolves #266

🧑‍💻 수행 작업

  • 과거 진행 그룹일 수 옵션 수정

📢 참고 사항

X

Summary by CodeRabbit

  • Chores
    • 테스트 데이터 매개변수가 업데이트되었습니다. 관련된 계산값들도 함께 갱신되었습니다.

Review Change Stack

@kelly6bf kelly6bf self-assigned this May 15, 2026
@coderabbitai

coderabbitai Bot commented May 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 1a0d1008-67ae-401e-91e9-25a769008aeb

📥 Commits

Reviewing files that changed from the base of the PR and between 185b694 and 96a2e85.

📒 Files selected for processing (1)
  • k6/common/test-data/test-data-common.js

📝 Walkthrough

Walkthrough

K6 테스트 데이터 파일에서 과거 그룹 실행 기간 상수를 3일에서 28일로 변경하여, 이에 따라 과거 활동 사이클 및 집계 계산값들이 새로운 기준으로 반영됩니다.

Changes

테스트 데이터 상수 업데이트

Layer / File(s) Summary
과거 활동 기간 상수 업데이트
k6/common/test-data/test-data-common.js
PAST_GROUP_RUNNING_DAY 상수가 3에서 28로 변경되며, 이를 기반으로 계산되는 PAST_TOTAL_ACTIVITY_CYCLE 및 과거 활동 관련 총량 파생 상수들이 새 기준으로 갱신됩니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • dogether-project/dogether-backend#232: 두 PR 모두 k6/common/test-data/test-data-common.js에서 과거 테스트 기간을 구성하는 상수 값을 변경하여 PAST_TOTAL_ACTIVITY_CYCLE 및 하위 PAST_TOTAL_* 날짜/집계 계산이 동일한 방식으로 연쇄 갱신됩니다.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항과 관련이 있으며, 과거 그룹 진행일 옵션 수정이라는 핵심 내용을 명확하게 전달합니다.
Description check ✅ Passed PR 설명이 템플릿의 필수 섹션을 모두 포함하고 있으며, 연관 이슈, 수행 작업, 참고 사항이 제공되었습니다.
Linked Issues check ✅ Passed PR의 변경사항(PAST_GROUP_RUNNING_DAY 상수 수정)이 이슈 #266의 k6 스크립트 수정 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 k6 테스트 데이터 상수 수정으로 이슈 #266의 범위 내에 있으며, 범위를 벗어난 변경은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch #-266

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kelly6bf kelly6bf merged commit 057916c into dev May 15, 2026
8 checks passed
@kelly6bf kelly6bf deleted the #-266 branch May 15, 2026 12:34
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.

[BE] K6 테스트 스크립트 수정

1 participant