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

Affects PMD Version: 6.30.0-SNAPSHOT

Rule:UseTryWithResources

Description:

From pmd/pmd#2871 (comment)

Code Sample demonstrating the issue:

https://chunk.io/pmd/fc7db65b2c6a46eca4c9a0c3012482e2/diff/checkstyle/index.html#A360

import j

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.

This code currently does not produce any violations:

class SomeExample(object):
    my_value = 1

    def get_value(self) -> int:
        return self.my_value

    def set_value(self, value: int):
        self.my_value = value

But, it should. We should check if there are two methods with get_ and set_ with the same name and raise a violation.

Related #1054
Rel

AngularJS Material uses gulp to build everything, so we're looking at using https://github.com/ivogabe/gulp-typescript for our builds. It seems like something could be built similar to the webpack plugin in order to have the types added during the Gulp pipeline.

It's not clear if typewiz-node could help with this already or not. We don't have a single main.ts to point to. Should we do somethi

There is a gap in the docs about managing your installation using github settings and imgbot settings

Main points:
For managing your imgbot installation check out github's docs.
They offer 2 options for managing this. You can select which repositories you want it installed in, or you can choose "all repositories including all future repositories".

You can log in and see your repos and req

let x = { default: 42}; // "default" is highlighted as keyword

undercover should create warnings on entire files that weren't required by specs and hence don't show up in the lcov output.

This can be done by changing logic starting from Undercover::Report#load_and_parse_file