Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow indented inline snapshots #16685

Merged
merged 3 commits into from
Jan 25, 2025
Merged

Conversation

pfgithub
Copy link
Contributor

What does this PR do?

Fixes #16403

Indentation in inline snapshots is ignored, and when new inline snapshots are created, they are automatically indented. Existing snapshots preserve indentation when updated.

@robobun
Copy link

robobun commented Jan 24, 2025

Updated 9:43 PM PT - Jan 23rd, 2025

@pfgithub, your commit 515606b has 1 failures in #10436:


🧪   try this PR locally:

bunx bun-pr 16685

@pfgithub pfgithub changed the title Pfg/allow indented inline snapshot Allow indented inline snapshots Jan 24, 2025
@pfgithub pfgithub marked this pull request as ready for review January 24, 2025 23:06
@Jarred-Sumner Jarred-Sumner merged commit 108b1c7 into main Jan 25, 2025
66 of 69 checks passed
@Jarred-Sumner Jarred-Sumner deleted the pfg/allow-indented-inline-snapshot branch January 25, 2025 02:13
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.

bun test toMatchInlineSnapshot fails due to backtick string indentation
3 participants