-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Fix inline snapshot indenting #9523
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9523 +/- ##
==========================================
+ Coverage 65.02% 65.03% +0.01%
==========================================
Files 283 283
Lines 12114 12113 -1
Branches 3000 3000
==========================================
+ Hits 7877 7878 +1
+ Misses 3598 3597 -1
+ Partials 639 638 -1
Continue to review full report at Codecov.
|
@@ -282,8 +282,6 @@ const createFormattingParser = ( | |||
return; | |||
} | |||
|
|||
snapshotMatcherNames.shift(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yay, no mutation 😀
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## master #9523 +/- ##
=======================================
Coverage 65.03% 65.03%
=======================================
Files 283 283
Lines 12113 12113
Branches 3000 3000
=======================================
Hits 7878 7878
Misses 3597 3597
Partials 638 638 Continue to review full report at Codecov.
|
Thanks! |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
Fixes #9477
Test plan
I've added a new test 🌮