sonar-scanner-npm icon indicating copy to clipboard operation
sonar-scanner-npm copied to clipboard

Published 20 hours ago • verified publisher icon bellingard

Allow to react and catch error when sonar analyisis fails

Open Floix opened this issue 5 years ago • 1 comments

Sonar analysis is part of our build pipeline. Sonar server may become unavailable at times, so I'd like to implement a retry mechanism (e.g., https://www.npmjs.com/package/retry). If the sonar analysis fails the process is killed and we cannot recover from this without complex (child-) process handling. Therefore, I'd like to propose to fail gracefully and provide an error object to the callback.

Let me know what you think.

Floix avatar Apr 26 '20 08:04 Floix

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Apr 26 '20 08:04 sonarqubecloud[bot]

Hi @Floix, are you still interested in this? Maybe you can update your branch and consider a default callback that preserves the previous behavior. WDT?

gabssnake avatar Sep 26 '22 17:09 gabssnake

Closing this as stale. Feel free to reopen if you want to try updating the PR and maintaining current functionality.

gabssnake avatar Oct 03 '22 17:10 gabssnake