Skip to content

Commit 9ec5454

Browse files
committed
fix: remove branching and explicit wording on s3
1 parent e3cd802 commit 9ec5454

File tree

2 files changed

+6
-13
lines changed

2 files changed

+6
-13
lines changed

samcli/commands/deploy/guided_context.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -182,8 +182,7 @@ def guided_prompts(self, parameter_override_keys):
182182

183183
click.echo("\n\tLooking for resources needed for deployment:")
184184
managed_s3_bucket = manage_stack(profile=self.profile, region=region)
185-
print_managed_s3_bucket_info(managed_s3_bucket, bold=True, show_s3_bucket_option=False)
186-
click.echo("\tAuto resolution of buckets can be turned off by setting resolve_s3=False")
185+
print_managed_s3_bucket_info(managed_s3_bucket)
187186

188187
image_repositories = (
189188
sync_ecr_stack(

samcli/lib/bootstrap/bootstrap.py

+5-11
Original file line numberDiff line numberDiff line change
@@ -36,26 +36,20 @@ def manage_stack(profile, region):
3636
return bucket_name
3737

3838

39-
def print_managed_s3_bucket_info(
40-
s3_bucket: str, bold: Optional[bool] = False, show_s3_bucket_option: Optional[bool] = True
41-
):
39+
def print_managed_s3_bucket_info(s3_bucket: str):
4240
"""
4341
Print information about the managed S3 bucket.
4442
4543
Parameters
4644
----------
4745
s3_bucket : str
4846
The name of the managed S3 bucket
49-
bold : bool, optional
50-
Whether to print the message in bold, by default False
51-
show_s3_bucket_option : bool, optional
52-
Whether to show the s3-bucket option message, by default True
5347
"""
5448
message = f"\n\tManaged S3 bucket: {s3_bucket}"
55-
click.secho(message, bold=bold)
56-
click.echo("\tA different default S3 bucket can be set in samconfig.toml")
57-
if show_s3_bucket_option:
58-
click.echo("\tOr by specifying --s3-bucket explicitly.")
49+
click.secho(message, bold=True)
50+
click.echo("\tAuto resolution of buckets can be turned off by setting resolve_s3=False")
51+
click.echo("\tTo use a specific S3 bucket, set --s3-bucket=<bucket_name>")
52+
click.echo("\tAbove settings can be stored in samconfig.toml")
5953

6054

6155
def get_current_account_id(profile: Optional[str] = None):

0 commit comments

Comments
 (0)