Skip to content

Conversation

@roger-zhangg
Copy link
Member

Which issue(s) does this change fix?

Why is this change necessary?

AppBuilder require region in sam config to detect deployed stack.
This change allow user to easier swtich to Appbuilder when deployed from SAM CLI

How does it address the issue?

Region is added to default samconfig.toml if user used
sam deploy --save-param

What side effects does this change have?

The Region is only populated but not read by sam cli

Mandatory Checklist

PRs will only be reviewed after checklist is complete

  • Add input/output type hints to new functions/methods
  • Write design document if needed (Do I need to write a design document?)
  • Write/update unit tests
  • Write/update integration tests
  • Write/update functional tests if needed
  • make pr passes
  • make update-reproducible-reqs if dependencies were changed
  • Write documentation

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 October 8, 2025 19:32
reedham-aws
reedham-aws previously approved these changes Oct 8, 2025
@roger-zhangg roger-zhangg added this pull request to the merge queue Oct 9, 2025
@roger-zhangg roger-zhangg removed this pull request from the merge queue due to a manual request Oct 9, 2025
@roger-zhangg roger-zhangg added this pull request to the merge queue Oct 9, 2025
Merged via the queue into aws:develop with commit a723b0f Oct 10, 2025
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants