Skip to content

docs(changelog): 남아 있던 병합 충돌 표시 삭제 - #364

Closed
WOULDU-pres wants to merge 1 commit into
devswha:mainfrom
WOULDU-pres:docs/changelog-conflict-marker
Closed

WOULDU-pres wants to merge 1 commit into
devswha:mainfrom
WOULDU-pres:docs/changelog-conflict-marker

Conversation

@WOULDU-pres

Copy link
Copy Markdown
Contributor

Change

안녕하세요. CHANGELOG.md의 Unreleased → Fixed 항목에 병합 충돌 표시 한 줄(||||||| 8c5c10d)이 그대로 남아 있어 삭제를 제안드립니다.

#351을 병합하는 과정(6a94148)에서 들어간 줄로 보입니다. 그 아래의 #342, #350 항목은 파일에서 이 위치에만 있는 정상 항목이라 그대로 두고, 표시 줄 하나만 지웠습니다. 저장소 전체에서 다른 충돌 표시는 찾지 못했습니다.

Validation

  • git grep -n -E '^(<<<<<<<|>>>>>>>|\|\|\|\|\|\|\|)( |$)': 수정 전에는 CHANGELOG.md:71 한 곳만 해당되었고, 수정 후에는 결과가 없습니다.
  • 변경은 CHANGELOG.md의 1줄 삭제뿐입니다.

확인해 주시면 감사하겠습니다.

@coderabbitai

coderabbitai Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: devswha/herdr-web-ui/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 3cf8be52-1505-4afd-9bf2-418ebe5b9e0e
📥 Commits

Reviewing files that changed from the base of the PR and between bb7ef50 and 4e74605.

📒 Files selected for processing (1)
  • CHANGELOG.md
💤 Files with no reviewable changes (1)
  • CHANGELOG.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Removed a conflict marker from the Unreleased changelog section.

Walkthrough

Removed a conflict marker from the Unreleased section of the changelog. The surrounding release notes remain unchanged.

Changes

Changelog update

Layer / File(s) Summary
Remove conflict marker
CHANGELOG.md
Removed the conflict marker between the OmO /new note and the Windows OmO note.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 4e746

This documentation-only cleanup removes a stray conflict marker from the changelog and has no effect on product behavior. It is safe to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the changelog change: removing a remaining merge-conflict marker.
Description check ✅ Passed The description includes both required sections. It explains the change and reports the validation command and results.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@devswha

devswha commented Oct 3, 2026

Copy link
Copy Markdown
Owner

확인해 주셔서 감사합니다. 지적하신 충돌 표시 줄(||||||| 8c5c10d)은 그 뒤 릴리스 준비 과정에서 main에서 이미 지워졌습니다. 지금 main의 CHANGELOG.md에는 충돌 표시가 남아 있지 않아, 이 PR은 더 반영할 내용이 없어 닫습니다.

@devswha devswha closed this Oct 3, 2026
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.

2 participants