From 6d32b65f2ec2e006918e17cdde30719a3978fc11 Mon Sep 17 00:00:00 2001
From: Kaizen Conroy <36202692+kaizencc@users.noreply.github.com>
Date: Thu, 8 Jun 2023 12:34:28 -0400
Subject: [PATCH] chore(app-staging-synthesizer): remove unnecessary
 documentation (#25896)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
---
 .../lib/default-staging-stack.ts                            | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/packages/@aws-cdk/app-staging-synthesizer-alpha/lib/default-staging-stack.ts b/packages/@aws-cdk/app-staging-synthesizer-alpha/lib/default-staging-stack.ts
index f630bebb50d84..d3db964441f44 100644
--- a/packages/@aws-cdk/app-staging-synthesizer-alpha/lib/default-staging-stack.ts
+++ b/packages/@aws-cdk/app-staging-synthesizer-alpha/lib/default-staging-stack.ts
@@ -150,9 +150,6 @@ export class DefaultStagingStack extends Stack implements IStagingResources {
    * Default asset publishing role name for file (S3) assets.
    */
   private get fileRoleName() {
-    // This role name can be a maximum of 64 letters. The reason why
-    // we slice the appId and not the entire name is because this.region
-    // can be a token and we don't want to accidentally cut it off.
     return `cdk-${this.appId}-file-role-${this.region}`;
   }
 
@@ -160,9 +157,6 @@ export class DefaultStagingStack extends Stack implements IStagingResources {
    * Default asset publishing role name for docker (ECR) assets.
    */
   private get imageRoleName() {
-    // This role name can be a maximum of 64 letters. The reason why
-    // we slice the appId and not the entire name is because this.region
-    // can be a token and we don't want to accidentally cut it off.
     return `cdk-${this.appId}-image-role-${this.region}`;
   }