rspec icon indicating copy to clipboard operation
rspec copied to clipboard

Create rule S7946: Logging statements should use lazy evaluation to avoid performance overhead

Open github-actions[bot] opened this issue 2 months ago • 5 comments

You can preview this rule here (updated a few minutes after each push).

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

github-actions[bot] avatar Sep 25 '25 12:09 github-actions[bot]

Quality Gate passed Quality Gate passed for 'rspec-tools'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonarqube-next[bot] avatar Sep 25 '25 12:09 sonarqube-next[bot]

Quality Gate passed Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonarqube-next[bot] avatar Sep 25 '25 12:09 sonarqube-next[bot]

I have rewritten the rule to make more sense sadly this rule does not apply to Python 3.14 anymore.

joke1196 avatar Oct 30 '25 13:10 joke1196

Quality Gate passed Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
1 Dependency risk
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonarqube-next[bot] avatar Oct 30 '25 14:10 sonarqube-next[bot]

Quality Gate passed Quality Gate passed for 'rspec-tools'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonarqube-next[bot] avatar Oct 30 '25 14:10 sonarqube-next[bot]

We decide to not implement such a rule as the value is too low and the risk of FP too high.

joke1196 avatar Nov 06 '25 13:11 joke1196