Skip to content
This repository has been archived by the owner on Oct 11, 2021. It is now read-only.

ci: run mypy on src folder #433

Merged
merged 1 commit into from
Jan 10, 2021
Merged

ci: run mypy on src folder #433

merged 1 commit into from
Jan 10, 2021

Conversation

redeboer
Copy link
Member

Fixes a bug introduced by #356: the src folder does not contain an __init__.py file, so mypy ignores it if you run mypy ..

The src folder does not a __init__.py, so mypy misses it
@redeboer redeboer added Bug Something isn't working 🖱️ DX Improvements to the Developer Experience labels Jan 10, 2021
@redeboer redeboer self-assigned this Jan 10, 2021
@codecov
Copy link

codecov bot commented Jan 10, 2021

Codecov Report

Merging #433 (02b620e) into master (7a98d6b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #433   +/-   ##
=======================================
  Coverage   90.07%   90.07%           
=======================================
  Files          22       22           
  Lines        3437     3437           
  Branches      847      847           
=======================================
  Hits         3096     3096           
  Misses        172      172           
  Partials      169      169           
Flag Coverage Δ
unittests 90.07% <ø> (ø)

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

@redeboer redeboer merged commit 38e575d into master Jan 10, 2021
@redeboer redeboer deleted the fix-tox-mypy branch January 10, 2021 21:21
redeboer added a commit to ComPWA/tensorwaves that referenced this pull request Jan 11, 2021
redeboer added a commit to ComPWA/tensorwaves that referenced this pull request Jan 11, 2021
redeboer added a commit to ComPWA/tensorwaves that referenced this pull request Jan 11, 2021
* ci: run mypy on src folder
  (see ComPWA/expertsystem#433)
* ci: upgrade developer dependencies
* fix: links to expertsystem API
* test: use fixture in test_angular_distributions
  Amplitude models are now generated on the fly and not read from disk anymore
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working 🖱️ DX Improvements to the Developer Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant