Skip to content

chore(action): Update automated-updates-to-sam-cli.yml to check if PR already exist#8621

Merged
roger-zhangg merged 2 commits intodevelopfrom
roger-zhangg-patch-2
Feb 3, 2026
Merged

chore(action): Update automated-updates-to-sam-cli.yml to check if PR already exist#8621
roger-zhangg merged 2 commits intodevelopfrom
roger-zhangg-patch-2

Conversation

@roger-zhangg
Copy link
Member

Which issue(s) does this change fix?

Why is this change necessary?

This workflow will still force push to change the commit even if the version change is already there. Causing unit test to run over and over again and therefore stopped at some point.

How does it address the issue?

This change will stop the workflow form executing if the PR already exist.

What side effects does this change have?

If a PR is updating to 1.107, and our latest version is 1.108, we need to manually close 1.107.

Mandatory Checklist

PRs will only be reviewed after checklist is complete

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

@roger-zhangg roger-zhangg requested a review from a team as a code owner February 2, 2026 19:29
@roger-zhangg roger-zhangg reopened this Feb 2, 2026
@roger-zhangg
Copy link
Member Author

Github Action outage, let's wait a bit

@roger-zhangg roger-zhangg added this pull request to the merge queue Feb 3, 2026
Merged via the queue into develop with commit b2ee6c9 Feb 3, 2026
42 checks passed
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.

3 participants