fix: do not declare _typecheck[_test] target when using tsc as transpiler

[jbedard]

Declaring the _typecheck[_test] targets is only required when tsc is not doing the typechecking while transpiling something.

We did add the _types target 100% of the time, transpiler(s) or not. I think we should keep that target 100% of the time. But when should the _typecheck[_test] targets be added? This reverts back to be more like before #705 where they only exist if tsc will not do type-checking in a different action.

Fix #719

---

Changes are visible to end-users: no

Test plan

• Covered by existing test cases
• New test cases added

[aspect-workflows]

Test

3 test targets passed

Targets

//examples/transpiler:custom_dts_transpiler-no_declarations-test [k8-fastbuild] 21ms
//examples/transpiler:custom_dts_transpiler-test [k8-fastbuild]                 26ms
//examples/transpiler:typecheck_fail-custom_transpilers_test [k8-fastbuild]     26ms

Total test execution time was 73ms. 97 tests (97.0%) were fully cached saving 8s.

---

Buildifier      Format

[jbedard]

So far voters are in favour of keeping this target which I also agree with, so converting to draft for now...