Skip to content

Mypy#106

Open
hanbinhu wants to merge 7 commits into
masterfrom
mypy
Open

Mypy#106
hanbinhu wants to merge 7 commits into
masterfrom
mypy

Conversation

@hanbinhu
Copy link
Copy Markdown
Collaborator

No description provided.

@hanbinhu hanbinhu requested a review from BichengYing March 19, 2022 01:19
Comment thread scripts/mypy.sh Outdated
# tools/mypy.sh [--version]

mypy_files=$(find bluefog examples test -name "*.py")
mypy --config-file=./scripts/.mypy.ini "$@" ${mypy_files} No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: new line

Comment thread scripts/.mypy.ini Outdated

[mypy-matplotlib.*,networkx.*,pytest.*,numpy.*]
follow_imports = silent
ignore_missing_imports = true No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: new line at end of file

Comment thread scripts/.mypy.ini Outdated
@@ -0,0 +1,5 @@
[mypy]

[mypy-matplotlib.*,networkx.*,pytest.*,numpy.*]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on my understanding, I think we can add more common libraries to .mypy.ini

[mypy-matplotlib.,networkx.,pytest.,numpy.,tensorflow.,torch.,psutil.*]
follow_imports = silent
ignore_missing_imports = true

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.

2 participants