-
Notifications
You must be signed in to change notification settings - Fork 340
Use built in sha1 for id generation #5081
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
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@Evangelink You are a code owner here, but I am just changing the underlying implementation and no tests were changed, and I also checked that the produced trx from old and new mstest SDK produces the same guids, so all should be good. |
I didn't know I was :D Thanks for the PR! Looking forward for the perf improvement. |
Description
Use built-in sha1 and slice for id generation.
Fix https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2050310
Fix microsoft/testfx#2762