Check string arguments with TR enabled

[realdadfish]

Check simple string arguments even when type resolution is enabled.
This prevented the UseRequire and UseCheckOrError checks find issues with TR enabled.

[realdadfish]

I'm not sure if / where this has to be described to get picked up by the changelog, or is this done automatically? Or should this just be merged?

[cortinico]

I'm not sure if / where this has to be described to get picked up by the changelog, or is this done automatically? Or should this just be merged?

It will be picked up automatically once we merge it 👍

[schalkms]

Thanks for creating this PR. I think that this change needs a corresponding unit test for regression purposes.

In my opinion detekt should require type resolving to be enabled in order to keep the complexity at a lower rate.

[codecov]

Codecov Report

Merging #2879 into master will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #2879   +/-   ##
=========================================
  Coverage     80.17%   80.17%           
  Complexity     2417     2417           
=========================================
  Files           421      421           
  Lines          7358     7358           
  Branches       1343     1343           
=========================================
  Hits           5899     5899           
  Misses          764      764           
  Partials        695      695           

Impacted Files	Coverage Δ	Complexity Δ
.../gitlab/arturbosch/detekt/rules/KtValueArgument.kt	66.66% <0.00%> (ø)	0.00 <0.00> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2de31f1...e022d0d. Read the comment docs.