-
Notifications
You must be signed in to change notification settings - Fork 749
[Automated] Update API Surface Area #13290
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
base: main
Are you sure you want to change the base?
Conversation
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 13290Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 13290" |
b1640c4 to
2ecc645
Compare
| public System.Threading.Tasks.ValueTask<string?> GetValueAsync(System.Threading.CancellationToken cancellationToken) { throw null; } | ||
| } | ||
|
|
||
| public sealed partial class PortAllocator : IPortAllocator |
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.
@davidfowl - does this need to be public? Or only the interface?
| System.Threading.Tasks.ValueTask<string?> IValueProvider.GetValueAsync(System.Threading.CancellationToken cancellationToken) { throw null; } | ||
| } | ||
|
|
||
| [System.Diagnostics.CodeAnalysis.Experimental("ASPIRECOMPUTE001", UrlFormat = "https://aka.ms/aspire/diagnostics/{0}")] |
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.
@captainsafia - I don't think these should reuse ASPIRECOMPUTE001. As that was already taken:
| public required ImagePullPolicy ImagePullPolicy { get { throw null; } set { } } | ||
| } | ||
|
|
||
| [System.Diagnostics.CodeAnalysis.Experimental("ASPIRECOMPUTE002", UrlFormat = "https://aka.ms/aspire/diagnostics#{0}")] |
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.
@captainsafia ASPIRECOMPUTE002 was already used for
| [Experimental("ASPIRECOMPUTE002", UrlFormat = "https://aka.ms/aspire/diagnostics/{0}")] | |
| ReferenceExpression GetHostAddressExpression(EndpointReference endpointReference) => throw new NotImplementedException(); |
72f1f26 to
d2eff89
Compare
d2eff89 to
04c84bf
Compare
Auto-generated update to the API surface to compare current surface vs latest release. This should only be merged once this surface area ships in a new release.