metricflow icon indicating copy to clipboard operation
metricflow copied to clipboard

Remove blanket "ignore missing imports" setting for mypy

Open tlento opened this issue 2 years ago • 0 comments

The current configuration for running mypy in the Metricflow .pre-commit-config.yaml uses the --ignore-missing-imports flag. This is generally undesirable.

This issue is to remove that and replace it with a combination of typed versions of those packages, type stubs, and more limited scope ignore missing import directives (either inline on the individual imports or else as part of the mypy.ini).

For more information on these approaches, please refer to https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports

tlento avatar Jul 28 '22 02:07 tlento