Skip to content

Conversation

pedrobsaila
Copy link
Contributor

Summary

Contributes to dotnet/runtime#109590

cc @ManickaP

@pedrobsaila pedrobsaila requested a review from a team as a code owner November 11, 2024 12:45
@ghost ghost added the area-System.Net label Nov 11, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Nov 11, 2024
Copy link
Contributor

Tagging subscribers to this area: @dotnet/ncl

@pedrobsaila pedrobsaila changed the title 109590 MailAddress should not accept email address with consecutive dots Nov 11, 2024
Copy link

Learn Build status updates of commit 14ed129:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Net.Mail/MailAddress.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@ManickaP
Copy link
Member

Sorry, I haven't thought this fully through, but the change is only for .NET next (.NET 10 - out Nov 2025 next year). We should keep the docs as-is for now. Also we need to keep the original description for previous versions and document that the change applies from .NET 10 onward.
So applying braking-change needs-breaking-change-doc-created label on the runtime PR should be enough for now.

Once again sorry and thank you for working on it.

@pedrobsaila
Copy link
Contributor Author

no worries @ManickaP

@pedrobsaila pedrobsaila deleted the 109590 branch November 11, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Net community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants