Skip to content

Conversation

@manuel3108
Copy link
Member

@manuel3108 manuel3108 commented Dec 6, 2025

This is an implementation detail. Will be helpful for a svelte print() related pr, that I will link in a moment (sveltejs/svelte#17319).

Let's assume this:

const context = ; // you get that from sowhere
const child_context = context.new();
child_context.newline();
context.append(child_context);

// before this pr
console.log(context.multiline); // false

// after this pr
console.log(context.multiline); // true

This is probably just an oversight from the 2.0 rework. The parent will always be a multiline-block as soon as it's children are. Apparently this is not covered by tests, no clue how to add a matching one.

@changeset-bot
Copy link

changeset-bot bot commented Dec 6, 2025

🦋 Changeset detected

Latest commit: bc699e2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
esrap Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 6, 2025

npm i https://pkg.pr.new/sveltejs/esrap@101

commit: bc699e2

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