Skip to content

fix(form_fit): let an ellipsis after a space start a line - #481

Merged
airmang merged 3 commits into
mainfrom
fix/form-fit-ellipsis-may-start-a-line
Oct 16, 2026
Merged

airmang merged 3 commits into
mainfrom
fix/form-fit-ellipsis-may-start-a-line

Conversation

@airmang

@airmang airmang commented Oct 16, 2026

Copy link
Copy Markdown
Owner

What

FormFit and the page estimate now start a line with an ellipsis (…) that follows a space, as Hancom Office lays it out. An ellipsis attached to the character before it still keeps to that character, as before; the existing script-symbol fixture holds that case.

Tests

A new Hancom-saved fixture (saved by Hancom Office 2024, scrubbed) holds tables of short sentences, one of whose cells breaks before "… ". Without the line caches it fails before this change and passes after.

🤖 Generated with Claude Code

FormFit and the page estimate now start a line with an ellipsis that follows a space, as Hancom Office lays it out. An ellipsis attached to the character before it still keeps to that character. A Hancom-saved fixture holds a table cell breaking so.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@airmang
airmang force-pushed the fix/form-fit-ellipsis-may-start-a-line branch from 5906b52 to 1f8357c Compare October 16, 2026 06:19

@airmang airmang left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

1f8357c 검토 완료. 줄 나눔 후보에서 공백 뒤 말줄임표만 허용하는 조건과 앞 글자에 붙은 말줄임표의 기존 규칙을 검토했습니다. 독립 체크아웃의 관련 테스트 12개가 통과했습니다. 해당 최신 head의 CI 전체 성공과 실제 Python 3.12 로그(6759 passed, 18 skipped, 1 xfailed; coverage 기준 및 Ruff·mypy·Pyright·공개 저장소 게이트 통과)를 확인했습니다. 검토 범위에서 재현된 코드 결함은 찾지 못했습니다. 다만 #475/#468/#472 병합 후 최신 main 0193c010218cb5c0abeed891be2aab4ec9de2f80과 CHANGELOG.md 내용 충돌이 발생함을 git merge-tree로 확인해 병합을 보류합니다. main의 변경 기록을 함께 보존하여 충돌을 해결한 뒤 새 head의 검사와 통합 결과를 다시 검증해야 합니다.

@airmang airmang left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

공백 뒤 말줄임표의 줄 시작 허용과 앞 글자에 붙은 말줄임표의 기존 금칙 처리를 검토했습니다. 충돌은 양쪽 기록과 테스트를 보존해 해결했고, 재배치 전후 함수·데코레이터·데이터의 의미가 같음을 대조했습니다. Python 3.10–3.14 전체 CI, CodeQL, 의존성 검사가 모두 성공했습니다. 같은 커밋의 중복 PR 테스트 실행은 취소하고 push 매트릭스 성공을 확인했습니다. 최종 통합본의 관련 테스트 1,692개도 통과했습니다. 검토한 head를 지정하여 squash 병합합니다.

@airmang
airmang merged commit bb790f5 into main Oct 16, 2026
8 of 13 checks passed
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.

1 participant