Skip to content

Warn when publishing isolated cells - #595

Merged
mdirolf merged 4 commits into
crosshare-org:masterfrom
idrivehtbusiness:fix/isolated-cell-publish-warning
Oct 2, 2026
Merged

mdirolf merged 4 commits into
crosshare-org:masterfrom
idrivehtbusiness:fix/isolated-cell-publish-warning

Conversation

@idrivehtbusiness

Copy link
Copy Markdown
Contributor

Fixes #593.
Adds detection for non-block cells that belong to neither an Across nor a Down entry and surfaces the condition through the existing publish warning flow.
Includes a regression test for an isolated cell surrounded by blocks.
This remains a warning rather than a publish error so constructors can still intentionally use isolated cells for gimmicks.

@mdirolf

mdirolf commented Oct 1, 2026

Copy link
Copy Markdown
Member

This looks good but can you fix the failing build?

@idrivehtbusiness

idrivehtbusiness commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor Author

hello! I’ve pushed a formatting fix for the failing check. The new CI run is currently awaiting approval before it can run.

@mdirolf
mdirolf merged commit b62ea13 into crosshare-org:master Oct 2, 2026
1 check passed
@mdirolf

mdirolf commented Oct 2, 2026

Copy link
Copy Markdown
Member

Thanks!

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.

Add publish warning for isolated cells

2 participants