Skip to content

Update manual CI workflows to take commit SHA#4078

Merged
aws-sdk-rust-ci merged 3 commits intomainfrom
landonxjames/update-ci
Apr 2, 2025
Merged

Update manual CI workflows to take commit SHA#4078
aws-sdk-rust-ci merged 3 commits intomainfrom
landonxjames/update-ci

Conversation

@landonxjames
Copy link
Contributor

@landonxjames landonxjames commented Apr 2, 2025

Motivation and Context

Description

Testing

Canary fails with appropriate error message if provided SHA and retrieved SHA (from the PR number) don't match: https://github.com/smithy-lang/smithy-rs/actions/runs/14210739747

Canary works if the provided SHA and the retrieved SHA do match: https://github.com/smithy-lang/smithy-rs/actions/runs/14210752482

Manual PR bot fails with appropriate error message if provided SHA and retrieved SHA do not match: https://github.com/smithy-lang/smithy-rs/actions/runs/14210797894

Manual PR bot works if the provided SHA and retrieved SHA do match: https://github.com/smithy-lang/smithy-rs/actions/runs/14210813156


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@landonxjames landonxjames marked this pull request as ready for review April 2, 2025 03:24
@landonxjames landonxjames requested review from a team as code owners April 2, 2025 03:24
@landonxjames landonxjames enabled auto-merge April 2, 2025 03:27
@aws-sdk-rust-ci aws-sdk-rust-ci disabled auto-merge April 2, 2025 03:31
@aws-sdk-rust-ci aws-sdk-rust-ci merged commit 5f849fd into main Apr 2, 2025
46 of 50 checks passed
@aws-sdk-rust-ci aws-sdk-rust-ci deleted the landonxjames/update-ci branch April 2, 2025 03:32
@github-actions
Copy link

github-actions bot commented Apr 2, 2025

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants