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.
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
Produce crashreport.json and use llvm-symbolizer to create stack trace #77578
Produce crashreport.json and use llvm-symbolizer to create stack trace #77578
Changes from all commits
ae22daa
770b4c5
b98f363
7f444a7
5cfff11
2988f89
bccea52
e804a12
b093dcb
f1caaaf
c39de18
f79abc2
8bc94d5
28a38b9
70545f6
f0b5df6
b4f5037
4504340
940c300
e40b789
230d239
09d2951
24d982d
d700447
e1d0fd9
12406ab
cb7e0cb
db9c273
8045668
5b567e0
f3700ec
5ce4de8
83235fd
1eae2fb
e0e3b7a
7daf27f
a91b8a8
fe39e5e
547fa85
5814640
c38743b
b7258ec
54e4eab
45f737f
afc62a1
fb3931f
a8170d3
e115e6b
8c92524
d9127cd
f75986c
63a07b7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
If elevating works on Mac right now, that's great, but I could see this not working forever, so we should give a little thought to what we would do instead.
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.
We should think of a way to do this smarter - timestamps don't work great with multiple tests running at the same time.
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.
I couldn't think of a better way. Let me know if we can do something so that we can get back the dump name or something.
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.
nit: we tend to use var only on cases where we know the type.
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.
not sure what you mean - Here, it is
IOrderedEnumerable<FileInfo>
and I just thoughtvar
would be shorter to write.