Add --includTargetType and --targetType options

[bz-canva]

I'm aware that it's recommended to consume the generated impacted target list with bazel query, as suggested in #173 and #151. However, our use case is that we do want to avoid the extra 30-60s spent on bazel query (we have a huge workspace).

This PR adds a --includeTargetType option to generate-hashes command, which will add "target type" to the generated JSON like this:

{
  "//cli:src/test/resources/fixture/integration-test-1.zip": "SourceFile#c259eba8539f4c14e4536c61975457c2990e090067893f4a2981e7bb5f4ef4cf",
  "//external:android_gmaven_r8": "Rule#795f583449a40814c05e1cc5d833002afed8d12bce5b835579c7f139c2462d61",
  "//cli:bazel-diff_deploy.jar": "GeneratedFile#4ae310f8ad2bc728934e3509b6102ca658e828b9cd668f79990e95c6663f9633",
  ...
}

Later, an extra --targetType=Rule can be used in get-impacted-targets command to filter out the targets with specific type.

This may be not what you like, but it does help us a lot so I'm contributing this from our fork.

The tests are updated accordingly.

[CLAassistant]

All committers have signed the CLA.

[tinder-maxwellelliott]

Thanks for doing this!

[tinder-maxwellelliott]

I believe this should be an array? You could desire to filter out for multiple types of target. Maybe this could also be backed by an enumeration to display the allowed values

[bz-canva]

Yeah, I have changed it to a Set to support multiple values. However, I was hesitant to change it to enum because in case of new types are added in the future, I expect the current code still work without need to add new values to the enum.

[tinder-maxwellelliott]

Thanks!

[bz-canva]

Thanks @tinder-maxwellelliott ! Do you mind releasing a new version so that we can use it directly from this repo?

[tinder-maxwellelliott]

Thanks @tinder-maxwellelliott ! Do you mind releasing a new version so that we can use it directly from this repo?

Done

[bz-canva]

Thank you sooooo much!