Many repositories need to fix, so please help if you like.

If you could help, it would be helpful if you could comment before starting the work not to overlapping.

Fix example

Run exit command after lint.

echo '::group:: Running golangci-lint with reviewdog 🐶 ...'
go

We have a problem: if files aren't loaded/required we don't have branch data. If they are added through track_files we give it 0/0 branches which we show as 100% coverage (all possible branches are covered).

That math is "wrong" though here because there are branches but we don't know what they are. We should probably count total branches here as "unkown" and establish that in our "math unive

Expected Behavior of the rule

The empty else block is reported:

                fun f() {
                    var i = 0
                    if (i == 0) {
                        println(i)
                    } else {
                        
                    }
                }

Context

EmptyIfBlock is only reporting empty then blocks, not empty else b

Affects PMD Version: 6.30.0-SNAPSHOT

Rule: UselessOverridingMethod

Description:

From pmd/pmd#2871 (comment)

Note: There seems to be a difference when having the class in the auxclasspath or not (typeresolution).

Code Sample demonstrating the issue:

And, ideally, a configurable list of annotations denoting generated source or AST structures.

See #105 for the kind of issue this seeks to avoid, and the limitations of this fix.

Rule request

I suggest adding comma , and dot . to the list of allowed exceptions for WPS226

Thesis

Forcing defining a constant for dot and comma strings worsen the code readability without need.

Reasoning

It is a classical pattern to build a sentence with th

I understand that optimisation of image/x-icon files has been added to Magick.NET (dlemstra/Magick.NET@c5e41c5). Are there any chances of this being added to Imgbot?

This has been requested before in #444, unfortunately, that issue has become stale without any comments whatsoever.