fix(bazel): reenable template type checking in ng_module (#34144)
due to an unfortunate condition in 168abc6d6f/packages/compiler-cli/src/ngtsc/program.ts (L430-L434)
the typechecking has been disabled when running under bazel + ivy.
As far as I can tell the ivyTemplateTypeCheck flag is now obsolete, so removing this
code from ng_module.bzl is desirable. I'll send a separate PR to remove the flag completely.
PR Close #34144
This commit is contained in:
parent
b8eb2f2f23
commit
d8792b3c36
|
@ -305,9 +305,6 @@ def _ngc_tsconfig(ctx, files, srcs, **kwargs):
|
|||
"enableSummariesForJit": is_legacy_ngc,
|
||||
"enableIvy": is_ivy_enabled(ctx),
|
||||
"fullTemplateTypeCheck": ctx.attr.type_check,
|
||||
# TODO(alxhub/arick): template type-checking for Ivy needs to be tested in g3 before it can
|
||||
# be enabled here.
|
||||
"ivyTemplateTypeCheck": False,
|
||||
# In Google3 we still want to use the symbol factory re-exports in order to
|
||||
# not break existing apps inside Google. Unlike Bazel, Google3 does not only
|
||||
# enforce strict dependencies of source files, but also for generated files
|
||||
|
|
Loading…
Reference in New Issue