mypy: Ignore redef in importlib_metadata hack and run mypy --platform win32

ni / nidaqmx-python

A Python API for interacting with NI-DAQmx

Other

428 stars 154 forks source link

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Fix two mypy issues uncovered in #512

nidaqmx/__init__.py is failing with error: Name "version" already defined (possibly by an import) [no-redef]
The GitHub workflow is only running mypy for Linux, not Windows

I don't know why we haven't seen this in other PRs. I can reproduce it with the master branch as of commit a60f369a3a8e874f541a23e4d71e2efebfa5b3a0

Why should this Pull Request be merged?

Fix mypy error.

What testing has been done?

None

Test Results

34 files ±0 34 suites ±0 43m 18s :stopwatch: -30s 2 244 tests ±0 1 908 :white_check_mark: ±0 336 :zzz: ±0 0 :x: ±0 39 650 runs ±0 34 114 :white_check_mark: ±0 5 536 :zzz: ±0 0 :x: ±0

Results for commit fe748fd4. ± Comparison against base commit a60f369a.

ni / nidaqmx-python