Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

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.

@github-actions github-actions bot added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Apr 11, 2025
@github-actions github-actions bot force-pushed the update-api-diffs branch 3 times, most recently from 7d0c9c2 to c0d5f4c Compare April 14, 2025 16:18
@github-actions github-actions bot force-pushed the update-api-diffs branch 2 times, most recently from a280eea to f1e17ab Compare April 16, 2025 16:18
@danmoseley danmoseley added the area-engineering-systems infrastructure helix infra engineering repo stuff label Apr 16, 2025
@github-actions github-actions bot force-pushed the update-api-diffs branch 6 times, most recently from cbed789 to 3752e74 Compare April 24, 2025 16:18
@github-actions github-actions bot force-pushed the update-api-diffs branch 7 times, most recently from 6fe8795 to a837a27 Compare May 2, 2025 16:17
@github-actions github-actions bot force-pushed the update-api-diffs branch 2 times, most recently from 9f37c81 to 4c75793 Compare May 4, 2025 16:16
public static ApplicationModel.IResourceBuilder<AzureUserAssignedIdentityResource> AddAzureUserAssignedIdentity(this IDistributedApplicationBuilder builder, string name) { throw null; }
}

public sealed partial class AzureUserAssignedIdentityResource : AzureProvisioningResource, IAppIdentityResource
Copy link
Member

Choose a reason for hiding this comment

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

Should it be named AzureUserAssignedManagedIdentityResource?

Copy link
Member

Choose a reason for hiding this comment

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

Azure.Provisioning calls it UserAssignedIdentity, so I don't think "Managed" is necessary.

FYI - @tg-msft

@joperezr
Copy link
Member

FWIW, since we talked about it yesterday, I have validated that all brand new packages/integrations we are adding this release are set to stay in preview. FYI: @DamianEdwards since you were asking about this.

@github-actions github-actions bot force-pushed the update-api-diffs branch from 139f642 to d4dfe99 Compare May 13, 2025 17:51
@github-actions github-actions bot force-pushed the update-api-diffs branch 2 times, most recently from 60be407 to 9c78d1c Compare May 15, 2025 16:18
@danmoseley
Copy link
Member

anything left here or can we close this now?

@joperezr
Copy link
Member

Let me rerun real quick against release branch to ensure the shape matches what we have in release/9.3 as opposed to main in case we have new apis in main already. After that I'll merge this in

@eerhardt
Copy link
Member

Let me rerun real quick against release branch to ensure the shape matches what we have in release/9.3 as opposed to main in case we have new apis in main already. After that I'll merge this in

FYI - [release/9.3] Respond to API Review feedback (dotnet/aspire#9285) was only in the release branch. I didn't make a PR for main, with the thinking that the release branch will merge back to main. So you will see that diff.

@github-actions github-actions bot force-pushed the update-api-diffs branch from 9c78d1c to 7b7eadb Compare May 15, 2025 21:06
@joperezr
Copy link
Member

Ok commit is pushed. @eerhardt can you confirm that this is showing what you expect that will be released in 9.3?

@joperezr
Copy link
Member

Doing a quick spot check it does seem like the changes are captured as expected.

@joperezr joperezr merged commit 19c5464 into main May 15, 2025
5 checks passed
@joperezr joperezr deleted the update-api-diffs branch May 15, 2025 21:32
@github-actions github-actions bot locked and limited conversation to collaborators Jun 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-engineering-systems infrastructure helix infra engineering repo stuff NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) Re-opened Github-Action PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants