Skip to content

Consolidate unix platforms flavors in S.N.Http#129926

Open
am11 wants to merge 5 commits into
dotnet:mainfrom
am11:feature/system.net.http/consolidate-unix
Open

Consolidate unix platforms flavors in S.N.Http#129926
am11 wants to merge 5 commits into
dotnet:mainfrom
am11:feature/system.net.http/consolidate-unix

Conversation

@am11

@am11 am11 commented Jun 27, 2026

Copy link
Copy Markdown
Member

@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Jun 27, 2026
@am11 am11 force-pushed the feature/system.net.http/consolidate-unix branch from 55d944f to f346587 Compare June 27, 2026 13:11
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@am11 am11 force-pushed the feature/system.net.http/consolidate-unix branch 2 times, most recently from 4f81283 to 62ba83e Compare June 27, 2026 13:13
@am11 am11 force-pushed the feature/system.net.http/consolidate-unix branch from 62ba83e to a9cc79e Compare June 27, 2026 13:18
[System.ComponentModel.EditorBrowsableAttribute(System.ComponentModel.EditorBrowsableState.Never)]
[System.ObsoleteAttribute("This API supports the .NET Framework infrastructure and is not intended to be used directly from your code.", true)]
public NetworkChange() { }
public static bool IsSupported { get { throw null; } }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new public API so it will need an API review.

LGTM otherwise

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please open API proposal issue?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#129943

(I can't find the thread but I read feature flag discussion that IsSupported is exempted from API reviews)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Net.Http 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.

4 participants