Merge pull request #275 from SpiNNakerManchester/use_build

github action fixes

.github/workflows/python_actions.yml

@@ -26,5 +26,5 @@ jobs:
       coverage-package: spinn_utilities
       flake8-packages: spinn_utilities unittests
       pylint-packages: spinn_utilities
-      mypy-packages: spinn_utilities
+      mypy-packages: spinn_utilities unittests
     secrets: inherit

setup.cfg

@@ -73,4 +73,5 @@ test =
         httpretty != 1.0.0
         types-appdirs
         types-requests
+        types-PyYAML
 

spinn_utilities/citation/citation_aggregator.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 import os
-import yaml  # type: ignore[import]
+import yaml
 import io
 import importlib
 import argparse

spinn_utilities/citation/citation_updater_and_doi_generator.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import yaml  # type: ignore[import]
+import yaml
 import io
 import requests
 import zipfile

unittests/citation/test_citation.py

@@ -16,7 +16,7 @@
 import tempfile
 import os
 import yaml
-import httpretty
+import httpretty  # type: ignore[import]
 from spinn_utilities.citation.citation_updater_and_doi_generator import (
     _Zenodo as Zenodo)
 

unittests/test_conf_loader.py

@@ -18,7 +18,7 @@
 import random
 import configparser
 import unittests  # CRITICAL: *THIS* package!
-from testfixtures import LogCapture
+from testfixtures import LogCapture  # type: ignore[import]
 import spinn_utilities.conf_loader as conf_loader
 import spinn_utilities.config_holder as config_holder
 from spinn_utilities.configs import (

unittests/test_log_checker.py

@@ -15,7 +15,7 @@
 import logging
 import pytest
 from spinn_utilities.log import FormatAdapter
-from testfixtures import LogCapture
+from testfixtures import LogCapture  # type: ignore[import]
 from spinn_utilities.testing import log_checker
 
 logger = FormatAdapter(logging.getLogger(__name__))

unittests/test_logger_utils.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 import logging
-from testfixtures import LogCapture
+from testfixtures import LogCapture  # type: ignore[import]
 import unittest
 from spinn_utilities import logger_utils
 from spinn_utilities.testing import log_checker

unittests/test_overrides.py

@@ -151,7 +151,7 @@ def test_undefaults_super_param():
         class Sub(Base):
             @overrides(Base.foodef)
             def foodef(self, x: Any, y: Any, z: Any) -> List[Any]:
-                return (z, y, x)
+                return [z, y, x]
     assert str(e.value) == BAD_DEFS
 
 
@@ -287,7 +287,7 @@ def bar(self):
         class Child(ParentTwo, ParentOne):
             @overrides(ParentTwo.foo)
             @overrides(ParentOne.foo)
-            def foo(self) -> int:
+            def foo(self) -> int:  # type: ignore
                 return 1
     except AttributeError:
         pass
@@ -311,7 +311,7 @@ def bar(self):
         class Child(ParentTwo, ParentOne):
             @overrides(ParentOne.foo)
             @overrides(ParentTwo.foo)
-            def foo(self) -> int:
+            def foo(self) -> int:  # type: ignore
                 return 1
     except AttributeError:
         pass

unittests/test_progress_bar.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 import pytest
-from testfixtures import LogCapture
+from testfixtures import LogCapture  # type: ignore[import]
 from spinn_utilities.config_setup import unittest_setup
 from spinn_utilities.progress_bar import (
     ProgressBar, DummyProgressBar, _EnhancedProgressBar as