Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 4 commits into from
Apr 2, 2024

Conversation

bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented Apr 2, 2024

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 a60f369

Why should this Pull Request be merged?

Fix mypy error.

What testing has been done?

None

Copy link

github-actions bot commented Apr 2, 2024

Test Results

    34 files  ±0      34 suites  ±0   43m 18s ⏱️ -30s
 2 244 tests ±0   1 908 ✅ ±0    336 💤 ±0  0 ❌ ±0 
39 650 runs  ±0  34 114 ✅ ±0  5 536 💤 ±0  0 ❌ ±0 

Results for commit fe748fd. ± Comparison against base commit a60f369.

@bkeryan bkeryan merged commit b73c3ac into master Apr 2, 2024
17 checks passed
@bkeryan bkeryan deleted the users/bkeryan/fix-mypy branch April 2, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants