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

Fix Mypy tests #10134

Merged
merged 1 commit into from
Sep 15, 2021
Merged

Fix Mypy tests #10134

merged 1 commit into from
Sep 15, 2021

Conversation

FlorianVeaux
Copy link
Member

@FlorianVeaux FlorianVeaux commented Sep 15, 2021

With mypy's bump, stubs for third-party libraries are not included anymore. They can be installed automatically by using --install-types and --non-interactive flags but unfortunately some stubs are now py3 only while we use mypy in --py2 mode.

To workaround the issue, this PR gives check the ability to pass additional style dependencies in each check tox.ini file.

@FlorianVeaux
Copy link
Member Author

Can be tested locally with ddev test -s datadog_checks_base aerospike azure_iot_edge cloud_foundry_api glusterfs ibm_mq marklogic mysql postgres rethinkdb snmp snowflake voltdb vsphere

@codecov
Copy link

codecov bot commented Sep 15, 2021

Codecov Report

Merging #10134 (ea28ba0) into master (982eb4a) will increase coverage by 0.06%.
The diff coverage is n/a.

Flag Coverage Δ
aerospike 86.97% <ø> (+0.36%) ⬆️
azure_iot_edge 82.01% <ø> (ø)
cloud_foundry_api 95.98% <ø> (+0.12%) ⬆️
datadog_checks_base 89.75% <ø> (+0.39%) ⬆️
datadog_checks_dev 78.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@FlorianVeaux FlorianVeaux merged commit aad7c43 into master Sep 15, 2021
@FlorianVeaux FlorianVeaux deleted the flo/fix_master_mypy_tests branch September 15, 2021 12:52
@ChristineTChen ChristineTChen changed the title Fix mypy tests Fix Mypy tests Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment