Skip to content

docs: initialize the input in the unflatten example - #213

Merged
Rich-Harris merged 1 commit into
sveltejs:mainfrom
Likio3000:docs/initialize-unflatten-input-20260923
Sep 23, 2026
Merged

Rich-Harris merged 1 commit into
sveltejs:mainfrom
Likio3000:docs/initialize-unflatten-input-20260923

Conversation

@Likio3000

Copy link
Copy Markdown
Contributor

The unflatten README example calls devalue.stringify(data) before declaring const data later in the same code block. Running the example in one scope therefore throws a ReferenceError before serialization.

Define a small input value to serialize and keep data for the revived result, with an expected-output comment to make the example self-contained.

Validation: static review of JavaScript lexical binding and the stringify/unflatten signatures in src/stringify.js and src/parse.js. This changes only the README; I did not run the project suite or install dependencies.

Prepared with AI assistance (OpenAI Codex), with source and diff review.

@changeset-bot

changeset-bot Bot commented Sep 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: dc16165

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@Rich-Harris Rich-Harris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks

@Rich-Harris
Rich-Harris merged commit 4760e1a into sveltejs:main Sep 23, 2026
5 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.

2 participants