Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix FP S3776 (cognitive-complexity): Ignore nested functions and default values #4520

Closed
ilia-kebets-sonarsource opened this issue Jan 24, 2024 · 1 comment · Fixed by SonarSource/eslint-plugin-sonarjs#444 or #4566
Assignees
Labels
mmf-2934 https://sonarsource.atlassian.net/browse/MMF-2934
Milestone

Comments

@ilia-kebets-sonarsource
Copy link
Contributor

ilia-kebets-sonarsource commented Jan 24, 2024

We wish to ignore:

  1. JavaScript's old way of default value declaration = parameter || literal:
    1. at declaration
    2. in assignment
  2. internal functions
@ilia-kebets-sonarsource ilia-kebets-sonarsource added the mmf-2934 https://sonarsource.atlassian.net/browse/MMF-2934 label Jan 24, 2024
@yassin-kammoun-sonarsource yassin-kammoun-sonarsource changed the title Fix FP S3776 (cognitive-complexity): Ignore internal functions and default values Fix FP S3776 (cognitive-complexity): Ignore nested functions and default values Feb 8, 2024
@yassin-kammoun-sonarsource
Copy link
Contributor

@ericmorand-sonarsource The PR is in another repository: eslint-plugin-sonarjs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment