Skip to content

Fix ProvisionalParagraph type typo - #1310

Merged
jorgemanrubia merged 3 commits into
mainfrom
fix-ProvisionalParagraph-type
Oct 25, 2026
Merged

jorgemanrubia merged 3 commits into
mainfrom
fix-ProvisionalParagraph-type

Conversation

@samuelpecher

Copy link
Copy Markdown
Collaborator
-    return this.config("provisonal_paragraph", {
+    return this.config("provisional_paragraph", {

😬

Copilot AI balanced review requested due to automatic review settings October 23, 2026 13:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Two gallery test-helper checks still use the old identifier and no longer recognize provisional paragraphs.

1 open issue
What changed in this PR

Corrects the internal type identifier for Lexxy’s provisional paragraph node.

Changes:

  • Renames "provisonal_paragraph" to "provisional_paragraph" in the node configuration.

[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

File Description
src/​nodes/​provisional_paragraph_node.js Corrects the provisional paragraph type spelling.

Review effort: Balanced

Comment thread src/nodes/provisional_paragraph_node.js

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approved

The narrowly scoped type correction and helper cleanup have no unresolved review issues.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@jorgemanrubia
jorgemanrubia merged commit d926002 into main Oct 25, 2026
12 checks passed
@jorgemanrubia
jorgemanrubia deleted the fix-ProvisionalParagraph-type branch October 25, 2026 12:57
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