Hello David,
Commented out code is based on some heuristics, which are not perfect. I understand that you are facing many false positives on this rule: Could you send more examples that trigger the rule? We then will be able to adjust the heuristic to no longer detect those patterns. Kind regards, |
Hi Dinesh, Our false positives are on the same pattern for now:
Regards, On Thu, Feb 12, 2015 at 5:18 PM, Dinesh Bolkensteyn <[hidden email]> wrote:
|
Hi, Any improvement planned on this rule? Another similar false positive: ![]() Thank you Regards, On Thu, Feb 12, 2015 at 5:26 PM, David Racodon <[hidden email]> wrote:
|
In reply to this post by Dinesh Bolkensteyn-2
Hi, Here follows another false positive example : * { 'xxXXX': 'AA0000011.000' } But the following lines do not raise any issue : * { 'xxXXX': 'AA0000011.000', 'copy': true } ... * {'xxXXX' : { 'yyy' : 'A0000001' }} ... * {'xxXXX' : { 'yyy' : ['0600000'], 'zzz' : ['123456A', '123456B', '123456C']}} Thanks in advance Michel |
Hello,
You can follow the resolution of this issue through the following ticket: SONARJS-388. Regards, Linda. On 4 May 2015 at 17:25, Michel Pawlak <[hidden email]> wrote: Hi, |
Hello,
Thanks for the ticket, however will handle the false positives I reported ? Kind regards, |
In reply to this post by Linda Martin
Thanks Linda. On Wed, May 6, 2015 at 5:40 PM, Linda Martin <[hidden email]> wrote:
|
In reply to this post by Michel Pawlak
Hello Michel,
Sorry for the late reply. I don't see how the code example you gave is a false-positive, it looks like it is indeed commented code. Maybe if you could give a larger code snippet including the whole comment I could understand the context. Thank in advance, Best regards, Linda. On 6 May 2015 at 18:23, Michel Pawlak <[hidden email]> wrote: Hello, |
Free forum by Nabble | Edit this page |