Skip to content

Conversation

@xinsong-cui
Copy link
Contributor

Issue #

Description of changes

Exclude SSO OIDC API validation to prevent preview build failures

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@xinsong-cui xinsong-cui marked this pull request as ready for review September 9, 2025 22:06
@xinsong-cui xinsong-cui requested a review from a team as a code owner September 9, 2025 22:06
@xinsong-cui xinsong-cui changed the title ignore ssoidc api check chore: ignore ssoidc api check Sep 9, 2025
@xinsong-cui xinsong-cui added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Sep 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 9, 2025

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

}
}
val packagesToIgnore = listOf(
"aws.sdk.kotlin.runtime.auth.credentials.internal.ssooidc",
Copy link
Member

Choose a reason for hiding this comment

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

Let's also ignore all the other internal-generated clients

@xinsong-cui xinsong-cui requested a review from lauzadis September 9, 2025 22:24
@xinsong-cui xinsong-cui changed the title chore: ignore ssoidc api check chore: ignore api check for internal generate clients Sep 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 9, 2025

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
pcs-jvm.jar closure 8,873,782 8,873,776 6 0.00%
iotsitewise-jvm.jar closure 12,897,970 12,898,479 -509 -0.00%
securityhub-jvm.jar closure 21,465,554 21,466,673 -1,119 -0.01%
securityhub-jvm.jar 13,527,494 13,528,614 -1,120 -0.01%
iotsitewise-jvm.jar 4,875,096 4,875,606 -510 -0.01%
sagemaker-jvm.jar closure 29,308,195 29,336,719 -28,524 -0.10%
sagemaker-jvm.jar 21,370,135 21,398,660 -28,525 -0.13%

@xinsong-cui xinsong-cui merged commit 4e1d264 into main Sep 9, 2025
23 checks passed
@xinsong-cui xinsong-cui deleted the ignore-api-check branch September 9, 2025 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants