Skip to content

Softbreak test#4813

Draft
JLouieS wants to merge 31 commits into
slab:mainfrom
JLouieS:softbreak-test
Draft

Softbreak test#4813
JLouieS wants to merge 31 commits into
slab:mainfrom
JLouieS:softbreak-test

Conversation

@JLouieS

@JLouieS JLouieS commented Jun 1, 2026

Copy link
Copy Markdown

No description provided.

@0xLeif

0xLeif commented Jul 23, 2026

Copy link
Copy Markdown

Review (via AI-assisted code review): REQUEST CHANGES

This PR is not mergeable in its current form.

  • 305 changed files, +10,329 lines: the bulk of the diff is checked-in build output under packages/quill/dist/ (LICENSE, README, .styl sources, SVG icons, compiled assets), plus a .gitignore edit. Committing generated dist artifacts contradicts the repo's own ignore rules and would pollute every future diff and release tarball.
  • The PR has no description and the title ("Softbreak test") plus branch name indicate this is an experiment, not a reviewed change. There is no explanation of what soft-break behavior is being changed or why.
  • With 31 commits touching hundreds of files, it is impossible to isolate or verify any actual source change that may be buried in here.

If there is a real soft-break fix in this branch, please open a new PR containing only the relevant source changes (no dist/), with a description of the bug, the fix, and tests. Closing this one seems appropriate.

@JLouieS
JLouieS marked this pull request as draft July 24, 2026 01:16
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.

3 participants