The example you provided [1] is a *correct *hint, again.
[1]
https://sonarcloud.io/project/issues?resolved=false=BUG=apache_ignite=AYdcaDNkmXUD5o3U1M1d
> Also, it is required to exclude from check examples, tests, jmh benchmarks
and so on.
-1 here. Examples and tests quality are also important.
>> No, I just think that we should remove 1 yaml file unless the sonar
check is not properly configured.
No, I just think that we should remove 1 yaml file unless the sonar check
is properly configured. -- corrected
чт, 28 сент. 2023 г. в 14:12, Ivan Daschinsky :
> No, I just think that we
No, I just think that we should remove 1 yaml file unless the sonar check
is not properly configured.
чт, 28 сент. 2023 г. в 14:09, Anton Vinogradov :
> Ivan,
> According to your proposal, should we also remove the "Platform .NET" tests
> [1] since they are always failed instead of fixing them?
I have already shown one example.
Here is another one --
https://sonarcloud.io/project/issues?resolved=false=BUG=apache_ignite=AYdcaDNkmXUD5o3U1M1d
Also, it is required to exclude from check examples, tests, jmh benchmarks
and so on. A lot of useless information doesn't help to find new issues in
Ivan,
According to your proposal, should we also remove the "Platform .NET" tests
[1] since they are always failed instead of fixing them?
[1]
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_PlatformNetCoreLinux?branch=%3Cdefault%3E=builds
On Thu, Sep 28, 2023 at 1:57 PM
> Sonar shows lots of false positives.
Thats not a true.
We have a *bad* code, unfortunatelly, and that's the *reason* of such hints.
Could you please show "a lot" of false positives?
The url you provided contains a lot of *correct* hints.
> But constantly red checks lead to ignoring of all
Sonar shows lots of false positives. We cannot add //NOSONAR to all of them
-- it is a tremendous job.
But constantly red checks lead to ignoring of all checks. Simply nobody pay
attention to them. It is not acceptable and contradicts to
the idea of running these checks.
чт, 28 сент. 2023 г. в
Removing a quality tool it not a good idea.
AFAIU, we have two issues here:
1) Sonar always failed at PR because ot token issues
Error: Failed to execute goal
org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar
(default-cli) on project apache-ignite: Project not found. Please check
Hi! It seems that these checks simply don't work, at least for PRs.
Yep, they work ok for master, but some of warnings from these tools seem to
be just rubbish, like
this one -- [1].
We should either do a tremendous job to fix these issues or simply disable
these checks.
Simply ignoring is not an