-
Notifications
You must be signed in to change notification settings - Fork 47k
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
Build Component Stacks from Native Stack Frames #18561
Merged
Merged
Commits on Apr 10, 2020
-
Configuration menu - View commit details
-
Copy full SHA for d0710d0 - Browse repository at this point
Copy the full SHA d0710d0View commit details -
In DEV it's somewhat likely that we'll see many logs that add component stacks. This could be slow so we cache the results of previous components.
Configuration menu - View commit details
-
Copy full SHA for 91b5f34 - Browse repository at this point
Copy the full SHA 91b5f34View commit details -
Fix special case when the function call throws in V8
In V8 we need to ignore the first line. Normally we would never get there because the stacks would differ before that, but the stacks are the same if we end up throwing at the same place as the control.
Configuration menu - View commit details
-
Copy full SHA for ffc3006 - Browse repository at this point
Copy the full SHA ffc3006View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0f2f008 - Browse repository at this point
Copy the full SHA 0f2f008View commit details -
This drops the requirement to include owner to pass the test.
Configuration menu - View commit details
-
Copy full SHA for 0575e9f - Browse repository at this point
Copy the full SHA 0575e9fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0b317f2 - Browse repository at this point
Copy the full SHA 0b317f2View commit details -
Add destructuring to force toObject which throws before the side-effects
This ensures that we don't double call yieldValue or advanceTime in tests. Ideally we could use empty destructuring but ES lint doesn't like it.
Configuration menu - View commit details
-
Copy full SHA for 66e67d5 - Browse repository at this point
Copy the full SHA 66e67d5View commit details -
Configuration menu - View commit details
-
Copy full SHA for df0ef46 - Browse repository at this point
Copy the full SHA df0ef46View commit details -
Configuration menu - View commit details
-
Copy full SHA for fbb0189 - Browse repository at this point
Copy the full SHA fbb0189View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.