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

tests: add basic type checking with mypy #756

Merged
merged 1 commit into from Sep 16, 2019

Conversation

@xginn8
Copy link
Collaborator

@xginn8 xginn8 commented Sep 9, 2019

Closes #736

Contributor checklist
  • Builds are passing
  • New tests have been added (for feature additions)
@xginn8 xginn8 requested review from aviau and sebito91 as code owners Sep 9, 2019
@aviau
Copy link
Member

@aviau aviau commented Sep 9, 2019

Nice!

@xginn8 xginn8 force-pushed the xginn8:736-mypy-typings branch 2 times, most recently from b2ba263 to 92ba134 Sep 10, 2019
Closes #736
@xginn8 xginn8 force-pushed the xginn8:736-mypy-typings branch from 92ba134 to c010f14 Sep 10, 2019
@xginn8
Copy link
Collaborator Author

@xginn8 xginn8 commented Sep 10, 2019

@aviau tests are now passing, PTAL as I'm new to mypy.

@aviau
Copy link
Member

@aviau aviau commented Sep 16, 2019

LGTM! The rest should be done in other PRs, type stuff as we go.

I have mypy experience so let me know if you guys have questions while typing stuff.

@aviau
aviau approved these changes Sep 16, 2019
@xginn8 xginn8 merged commit f749764 into influxdata:master Sep 16, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@xginn8 xginn8 deleted the xginn8:736-mypy-typings branch Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.