-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Merging internal commits for release/6.0 #67928
Merging internal commits for release/6.0 #67928
Conversation
This add validation for embedded newlines in email addresses. Based on https://dev.azure.com/dnceng/internal/_git/dotnet-runtime/pullrequest/20738 There is opt-in System.Net.Mail.EnableFullDomainLiterals switch to allow previous behavior
…-merge-6.0-2022-04-12-1157
Tagging subscribers to this area: @dotnet/ncl Issue Detailsnull
|
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.
@vseanreesermsft I assume this is all scripted so it's not weird that all the commits have the same name, and the end result is fewer changed files than the total files changes by all the commits. Correct?
I ask because your other internal merge PRs have unique names in the commits.
Tagging subscribers to this area: @dotnet/runtime-infrastructure Issue Detailsnull
|
I'll open issues for the failures:
SSL test throws "unstable on Windows 11"
Timeout in System.Text.Json
And a bunch of authentication failures
|
In the first one, we should skip not fail. RemoteExecutor does not know what to do with SkipTestExecution. Ideally it could catch it and "tunnel" it through to rethrow in the parent -- would be a bug in dotnet/arcade. Or, we should never throw it within the code running remotely - a test bug. |
No description provided.