chore(eslint): remove unneccessary semicolon #32724
Conversation
github-actions
bot
added
p2
repeat-contributor
| }; | ||
| } |
There was a problem hiding this comment.
There seems to be no eslint rule regarding semicolons in interface definitions, but the overwhelming majority of interface definitions in this file do not have semicolons.
Therefore, we believe that this semicolon should also be removed.
There was a problem hiding this comment.
If this modification is not needed, I'll remove it.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #32724 +/- ##
=======================================
Coverage 81.52% 81.52%
=======================================
Files 222 222
Lines 13715 13715
Branches 2417 2417
=======================================
Hits 11181 11181
Misses 2254 2254
Partials 280 280
Flags with carried forward coverage won't be shown. Click here to find out more.
|
aws-cdk-automation
added
the
pr/needs-community-review
| @@ -927,7 +927,7 @@ export class UserPool extends UserPoolBase { | |||
|
|
|||
| const userPoolId = arnParts.resourceName; | |||
| // ex) cognito-idp.us-east-1.amazonaws.com/us-east-1_abcdefghi | |||
| const providerName = `cognito-idp.${arnParts.region}.${Stack.of(scope).urlSuffix}/${userPoolId}`;; | |||
| const providerName = `cognito-idp.${arnParts.region}.${Stack.of(scope).urlSuffix}/${userPoolId}`; | |||
There was a problem hiding this comment.
👍 There are 18 occurrences of ;; in the project at the moment. It would be nice to adjust the other 17 as well.
There was a problem hiding this comment.
would be similarly amazing to figure out why its happening and/or add a linter rule against it
There was a problem hiding this comment.
@lpizzinidev @kaizencc
added @stylistic/js/no-extra-semi rule.
aws-cdk-automation
removed
the
pr/needs-community-review
yasuaki640
changed the title
yasuaki640
force-pushed
the
remove-uneccessary-semicolon
branch
from
8476a31 to
42d501b
Compare
yasuaki640
requested a deployment
to
test-pipeline
GitHub Actions
Waiting
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Issue # (if applicable)
Closes N/A
Reason for this change
I would like to reduce noise during development by reducing the number of warnings output by the IDE.
Description of changes
Removed unneccessary semicolon.
Describe any new or updated permissions being added
<!— What new or updated IAM permissions are needed to support the changes being introduced ? -->
Description of how you validated changes
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license