ignore_type_checking_imports default value is false but documentation says it's true

[DetachHead]

https://docs.gauge.sh/usage/faq#are-conditional-imports-checked

here it says the following:

Tach will check all imports in your source files, including those which are called conditionally. The only exceptions are imports made within TYPE_CHECKING conditional blocks. If you want to disable checks for these imports, you can add ignore_type_checking_imports: true to your tach.toml.

however i had to add ignore_type_checking_imports = false to my tach.toml to get it to report TYPE_CHECKING imports, so it seems that it's actually true by default.

i'm not sure which one is intended, but i believe false by default is the way to go, to reduce the trisk of users not knowing about the option and not being made aware of TYPE_CHECKING imports that violate their rules. imo strict by default is always better

[emdoyle]

The docs have been corrected, thanks for pointing this out! The current default value is true, but the docs you quoted suggested that it was false. It's also confusing because of the negation in the name of the config value.

I'm open to switching to the strict default, but I would also want to identify those imports in the output of tach check, and provide guidance that this config exists.

I really appreciate your patience on this issue and the others you've raised. We've just shipped several heavy PRs and should now be able to resolve those issues later this week (we also have some older issues to return to).