ishangsf/rspec
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Modify rule S107: Do not count TypeScript parameter properties (#2898)

Browse Source 
## Review

A dedicated reviewer checked the rule description successfully for:

- [ ] logical errors and incorrect information
- [ ] information gaps and missing content
- [ ] text style and tone
- [ ] PR summary and labels follow [the
guidelines](https://github.com/SonarSource/rspec/#to-modify-an-existing-rule)
This commit is contained in:
Yassin Kammoun 2023-08-18 14:01:14 +02:00 committed by GitHub
parent 9ce1a8de8e
commit 289a124d78
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
rules/S107/javascript/rule.adoc
View File

@ -4,17 +4,17 @@ include::../rule.adoc[]
=== Exceptions === Exceptions
The rule ignores constructors where parameters are *all* parameter properties: The rule ignores TypeScript parameter properties when counting parameters:
[source,javascript] [source,javascript]
---- ----
class C { class C {
constructor( constructor(
private param1: number, private param1: number, // ignored
private param2: boolean, param2: boolean, // counted
private param3: string, public param3: string, // ignored
private param4: string[], readonly param4: string[], // ignored
private param5: number | string param5: number | string // counted
) {} // Compliant by exception ) {} // Compliant by exception
} }
---- ----