Skip to content

chore: host_university 컬럼 제약 조건 수정#784

Merged
whqtker merged 2 commits into
developfrom
refactor/extend-format-name-column-length
Jun 22, 2026
Merged

chore: host_university 컬럼 제약 조건 수정#784
whqtker merged 2 commits into
developfrom
refactor/extend-format-name-column-length

Conversation

@whqtker

@whqtker whqtker commented Jun 22, 2026

Copy link
Copy Markdown
Member

관련 이슈

작업 내용

특이 사항

리뷰 요구사항 (선택)

@whqtker whqtker self-assigned this Jun 22, 2026
@whqtker whqtker added 리팩터링 진행 중 자유롭게 merge 가능 labels Jun 22, 2026
@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@whqtker, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 56 minutes and 5 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 61166945-a19a-43c4-adfe-76a5df64c91f

📥 Commits

Reviewing files that changed from the base of the PR and between b5109ac and edf62ac.

📒 Files selected for processing (1)
  • src/main/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoRowSaver.java

Walkthrough

이번 PR은 두 가지 파일에 걸친 작은 하지만 중요한 변경입니다.

  1. 엔티티 컬럼 길이 수정

    • HostUniversity.javaformatName 필드에 선언된 @Column(name = "format_name")length 값이 100에서 200으로 변경되었습니다.
  2. Flyway 마이그레이션 스크립트 추가

    • V55__extend_host_university_format_name_column.sql 파일이 새로 추가되었으며, host_university 테이블의 format_name 컬럼을 VARCHAR(200) NOT NULL로 변경하는 ALTER TABLE ... MODIFY COLUMN 구문이 포함되어 있습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive 필수 섹션인 '관련 이슈'는 작성되었으나, '작업 내용'과 '특이 사항' 섹션이 미작성 상태입니다. PR 설명에서 '작업 내용' 섹션을 작성하여 변경사항을 구체적으로 설명해주세요. formatName 컬럼의 길이 변경 사유와 영향 범위를 명시하면 좋습니다.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 설명하고 있습니다. 'host_university 컬럼 제약 조건 수정'은 정확하고 간결합니다.
Linked Issues check ✅ Passed PR의 코드 변경사항(formatName 길이 100→200)이 이슈 #783의 'host_university 컬럼 글자 수 수정' 목표와 일치합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 host_university의 formatName 컬럼 제약 조건 수정이라는 범위 내에서 이루어졌습니다.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/extend-format-name-column-length

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b5109ac7cb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@src/main/java/com/example/solidconnection/university/domain/HostUniversity.java`:
- Line 34: The formatName column length in HostUniversity class has been
expanded to 200 characters, but the corresponding input validation in
AdminUnivApplyInfoRowSaver class still truncates universityFormatName to 100
characters using applyWithLength(value, 100, ...). Update the applyWithLength
method call in the universityFormatName handling section of
AdminUnivApplyInfoRowSaver to use 200 instead of 100 to match the column length
definition in HostUniversity and ensure the full expanded length is preserved
during the import process.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c04efd9b-509e-4da6-8759-8d83916c29e6

📥 Commits

Reviewing files that changed from the base of the PR and between d808166 and b5109ac.

📒 Files selected for processing (2)
  • src/main/java/com/example/solidconnection/university/domain/HostUniversity.java
  • src/main/resources/db/migration/V55__extend_host_university_format_name_column.sql

@whqtker whqtker merged commit 3144296 into develop Jun 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

리팩터링 진행 중 자유롭게 merge 가능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: host_university 컬럼 글자 수 수정

1 participant