Skip to content

Commit 6d32b65

Browse files
authored
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*
1 parent 445543c commit 6d32b65

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

packages/@aws-cdk/app-staging-synthesizer-alpha/lib/default-staging-stack.ts

-6
Original file line numberDiff line numberDiff line change
@@ -150,19 +150,13 @@ export class DefaultStagingStack extends Stack implements IStagingResources {
150150
* Default asset publishing role name for file (S3) assets.
151151
*/
152152
private get fileRoleName() {
153-
// This role name can be a maximum of 64 letters. The reason why
154-
// we slice the appId and not the entire name is because this.region
155-
// can be a token and we don't want to accidentally cut it off.
156153
return `cdk-${this.appId}-file-role-${this.region}`;
157154
}
158155

159156
/**
160157
* Default asset publishing role name for docker (ECR) assets.
161158
*/
162159
private get imageRoleName() {
163-
// This role name can be a maximum of 64 letters. The reason why
164-
// we slice the appId and not the entire name is because this.region
165-
// can be a token and we don't want to accidentally cut it off.
166160
return `cdk-${this.appId}-image-role-${this.region}`;
167161
}
168162

0 commit comments

Comments
 (0)