-
Notifications
You must be signed in to change notification settings - Fork 29
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
Update submodule to latest master
in microsoft/main
#1383
Conversation
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.
Thanks! Auto-approving.
ef2fc33
to
5c9b2d6
Compare
Please don't resolve this conflicts until all my SymCrypt PRs are merged. |
5c9b2d6
to
5b7c726
Compare
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.
^
87a909e
to
0f15d0e
Compare
0f15d0e
to
4f1bf64
Compare
CI is failing due to golang/go#70463. I've already sent a fix for it: https://go-review.googlesource.com/c/go/+/630096. |
Pull request was closed
2f65945
to
ee6a6bf
Compare
Patch conflicts resolved, now I only have to fix the tests... |
For #1416. |
The fixes necessary to reconcile our fork with upstream are growing to the point that the PR is hard to review. I'll be extracting some of the commits to their own PR. |
7c4ee0a
to
2ba8745
Compare
fb10acd
to
d2ea31c
Compare
Finally, no conflicts and tests passing 🎉. |
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.
Let's merge it!
Hi! I'm a bot, and this is an automatically generated upstream sync PR. 🔃
After submitting the PR, I will attempt to enable auto-merge in the "merge commit" configuration.
For more information, visit sync documentation in microsoft/go-infra.