Skip to content

Commit 1e2c1ae

Browse files
authored
chore(eslint): enforce spacing at beginning of comments (#33077)
enforces spacing at beginning of comments. we are exempting situations that have `///` because they are special comments that jsii recognizes, and situations that have `////////` to delineate a split of some sport. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 6a9cbc2 commit 1e2c1ae

File tree

90 files changed

+365
-360
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

90 files changed

+365
-360
lines changed

packages/@aws-cdk-testing/cli-integ/tests/cli-integ-tests/cli.integtest.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,8 @@ integTest('doubly nested stack',
361361
await fixture.cdkDeploy('with-doubly-nested-stack', {
362362
captureStderr: false,
363363
});
364-
}));
364+
}),
365+
);
365366

366367
integTest(
367368
'nested stack with parameters',
@@ -404,7 +405,7 @@ integTest(
404405
);
405406
expect(response.Stacks?.[0].StackStatus).toEqual('REVIEW_IN_PROGRESS');
406407

407-
//verify a change set was created with the provided name
408+
// verify a change set was created with the provided name
408409
const changeSetResponse = await fixture.aws.cloudFormation.send(
409410
new ListChangeSetsCommand({
410411
StackName: stackArn,

packages/@aws-cdk-testing/framework-integ/test/aws-ec2/test/integ.bastion-host-disable-al2023-feature-flag.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const app = new cdk.App();
1414
class TestStack extends cdk.Stack {
1515
constructor(scope: cdk.App, id: string, props?: cdk.StackProps) {
1616
super(scope, id, props);
17-
this.node.setContext(BASTION_HOST_USE_AMAZON_LINUX_2023_BY_DEFAULT, false); //disable feature flag
17+
this.node.setContext(BASTION_HOST_USE_AMAZON_LINUX_2023_BY_DEFAULT, false); // disable feature flag
1818

1919
const vpc = new ec2.Vpc(this, 'VPC');
2020

packages/@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.loggroupclass.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ const describeInfrequentAccess = integTest.assertions.awsApiCall('CloudWatchLogs
5151
logGroupNamePrefix: logGroupInfrequentAccess.logGroupName,
5252
});
5353

54-
//valdiate
54+
// valdiate
5555
describeInfrequentAccess.expect(ExpectedResult.objectLike({
5656
logGroups: [
5757
{

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/bedrock/integ.invoke-model.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ const prompt3 = new BedrockInvokeModel(stack, 'Prompt3', {
7171
});
7272

7373
/** Test for Bedrock s3 URI Path */
74-
//State Machine Execution will fail for the following input as it expects a valid s3 URI from previous prompt
74+
// State Machine Execution will fail for the following input as it expects a valid s3 URI from previous prompt
7575
const prompt4 = new BedrockInvokeModel(stack, 'Prompt4', {
7676
model,
7777
input: { s3InputUri: '$.names' },
@@ -144,7 +144,7 @@ const llamaPrompt3 = new BedrockInvokeModel(stack, 'LlamaPrompt3', {
144144
});
145145

146146
/** Test for Bedrock s3 URI Path */
147-
//State Machine Execution will fail for the following input as it expects a valid s3 URI from previous prompt
147+
// State Machine Execution will fail for the following input as it expects a valid s3 URI from previous prompt
148148
const llamaModel4 = bedrock.FoundationModel.fromFoundationModelId(stack, 'LlamaModel4', bedrock.FoundationModelIdentifier.META_LLAMA_3_2_90B_INSTRUCT_V1);
149149
const llamaPrompt4 = new BedrockInvokeModel(stack, 'LlamaPrompt4', {
150150
model: llamaModel4,

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions/test/integ.state-machine-and-activity-cmk.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ const testCase = new IntegTest(app, 'StateMachineAndActivityWithCMKEncryptionCon
4747
testCases: [stack],
4848
});
4949

50-
//Start execution
50+
// Start execution
5151
const start = testCase.assertions.awsApiCall('StepFunctions', 'startExecution', {
5252
stateMachineArn: stack.stateMachine.stateMachineArn,
5353
});

packages/@aws-cdk-testing/framework-integ/test/core/test/integ.cfn-mappings-find-in-map.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { IntegTest } from '@aws-cdk/integ-tests-alpha';
44

55
const app = new cdk.App();
66

7-
const stack = new cdk.Stack(app, 'core-cfn-mapping-1'/*,{ env }*/);
7+
const stack = new cdk.Stack(app, 'core-cfn-mapping-1'/* ,{ env }*/);
88

99
const backing = {
1010
TopLevelKey1: {

packages/@aws-cdk/aws-ec2-alpha/lib/ipam.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ class IpamPool extends Resource implements IIpamPool {
344344
throw new Error('awsService is required when addressFamily is set to ipv6');
345345
}
346346

347-
//Add tags to the IPAM Pool if name is provided
347+
// Add tags to the IPAM Pool if name is provided
348348
if (props.ipamPoolName) {
349349
Tags.of(this).add(NAME_TAG, props.ipamPoolName);
350350
}

packages/@aws-cdk/aws-ec2-alpha/lib/route.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -720,7 +720,7 @@ export class Route extends Resource implements IRouteV2 {
720720
this.destination = props.destination;
721721
const isDestinationIpv4 = NetworkUtils.validIp(props.destination);
722722
if (!isDestinationIpv4) {
723-
//TODO Validate for IPv6 CIDR range
723+
// TODO Validate for IPv6 CIDR range
724724
this.destinationIpv6Cidr = props.destination;
725725
} else {
726726
this.destinationIpv4Cidr = props.destination;
@@ -742,7 +742,7 @@ export class Route extends Resource implements IRouteV2 {
742742
}
743743
this.node.defaultChild = this.resource;
744744

745-
//Create a route only after target gateway or endpoint is created
745+
// Create a route only after target gateway or endpoint is created
746746
if (this.target.gateway) {
747747
this.node.addDependency(this.target.gateway);
748748
}

packages/@aws-cdk/aws-ec2-alpha/lib/subnet-v2.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ export class SubnetV2 extends Resource implements ISubnetV2 {
259259

260260
overlap = validateOverlappingCidrRanges(props.vpc, props.ipv4CidrBlock.cidr);
261261

262-
//check whether VPC supports ipv6
262+
// check whether VPC supports ipv6
263263
if (props.ipv6CidrBlock?.cidr) {
264264
validateSupportIpv6(props.vpc);
265265
overlapIpv6 = validateOverlappingCidrRangesipv6(props.vpc, props.ipv6CidrBlock?.cidr);

packages/@aws-cdk/aws-ec2-alpha/lib/util.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/*eslint no-bitwise: ["error", { "allow": ["~", "|", "<<", "&"] }] */
1+
/* eslint no-bitwise: ["error", { "allow": ["~", "|", "<<", "&"] }] */
22

33
import { ISubnet } from 'aws-cdk-lib/aws-ec2';
44

0 commit comments

Comments
 (0)