Dev Tool Updates

.gitignore

@@ -1,11 +1,12 @@
 # Distribution / Packaging / Build files
-.coverage
+.coverage*
 doc/build/*
 doc/source/ait.core.test*
 doc/source/ait.core.server.test*
 build/lib/*
 *.egg-info/
 *.egg
+.tox
 
 # Object Files
 *.o
@@ -40,6 +41,7 @@ src/doc/dict/cmd/cmddict-07-cmddefs.tex
 
 # Temporary files
 *~
+tags*
 
 # Unignore (hack)
 !build/clean.py

.pre-commit-config.yaml

@@ -0,0 +1,85 @@
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.0.1
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-merge-conflict
+    -   id: debug-statements
+
+-   repo: https://github.com/asottile/reorder_python_imports
+    rev: v2.6.0
+    hooks:
+    -   id: reorder-python-imports
+        files: ^src/|test/
+
+-   repo: local
+    hooks:
+    -   id: mypy
+        name: mypy
+        entry: mypy ait
+        language: system
+        pass_filenames: false
+        exclude: "bin/"
+
+-   repo: local
+    hooks:
+    -   id: black
+        name: black
+        entry: black
+        files: ^src/|test/
+        language: system
+        types: [python]
+
+-   repo: local
+    hooks:
+    -   id: flake8
+        name: flake8
+        entry: flake8 ait
+        language: system
+        pass_filenames: false
+
+-   repo: local
+    hooks:
+    -   id: tlm_yaml_check
+        name: AIT TLM YAML check
+        entry: 'ait-yaml-validate --tlm'
+        language: system
+        stages: [push]
+        pass_filenames: false
+
+-   repo: local
+    hooks:
+    -   id: cmd_yaml_check
+        name: AIT CMD YAML check
+        entry: 'ait-yaml-validate --cmd'
+        language: system
+        stages: [push]
+        pass_filenames: false
+
+-   repo: local
+    hooks:
+    -   id: evr_yaml_check
+        name: AIT EVR YAML check
+        entry: 'ait-yaml-validate --evr'
+        language: system
+        stages: [push]
+        pass_filenames: false
+
+-   repo: local
+    hooks:
+    -   id: limits_yaml_check
+        name: AIT LIMITS YAML check
+        entry: 'ait-yaml-validate --limits'
+        language: system
+        stages: [push]
+        pass_filenames: false
+
+-   repo: local
+    hooks:
+    -   id: tests
+        name: Tests
+        entry: pytest
+        language: system
+        stages: [push]
+        pass_filenames: false

MANIFEST.in

@@ -0,0 +1 @@
+include ait/core/data/*.json

README.rst

@@ -5,7 +5,7 @@ The AMMOS Instrument Toolkit (Formerly the Bespoke Links to Instruments
 for Surface and Space (BLISS)) is a Python-based software suite
 developed to handle Ground Data System (GDS), Electronic Ground Support
 Equipment (EGSE), commanding, telemetry uplink/downlink, and sequencing
-for instrument and CubeSat Missions. It is a generalization and expansion 
+for instrument and CubeSat Missions. It is a generalization and expansion
 of tools developed for a number of ISS
 missions.
 
@@ -39,7 +39,7 @@ Guide <https://github.com/NASA-AMMOS/AIT-Core/wiki/Contributing>`__.
 |docs|
 
 .. |travis| image:: https://travis-ci.com/NASA-AMMOS/AIT-Core.svg?branch=master
-    :target: https://travis-ci.com/NASA-AMMOS/AIT-Core 
+    :target: https://travis-ci.com/NASA-AMMOS/AIT-Core
 
 .. |docs| image:: https://readthedocs.org/projects/ait-core/badge/?version=master
     :alt: Documentation Status

ait/__init__.py

@@ -12,4 +12,4 @@
 # or other export authority as may be required before exporting such
 # information to foreign countries or providing access to foreign persons.
 
-__import__('pkg_resources').declare_namespace(__name__)
+__import__("pkg_resources").declare_namespace(__name__)

ait/core/__init__.py

@@ -14,31 +14,34 @@
 
 import sys
 
-from ait.core import cfg, log
+# cfg isn't used but we want the AIT-level config attribute created
+from ait.core import cfg, log  # noqa
 
 
 def deprecated(message):
-  def deprecated_decorator(func):
-      def deprecated_func(*args, **kwargs):
-          log.warn("{} is a deprecated function. {}".format(func.__name__, message))
-          return func(*args, **kwargs)
-      return deprecated_func
-  return deprecated_decorator
+    def deprecated_decorator(func):
+        def deprecated_func(*args, **kwargs):
+            log.warn("{} is a deprecated function. {}".format(func.__name__, message))
+            return func(*args, **kwargs)
 
-sys.modules['ait'].deprecated = deprecated
-sys.modules['ait'].DEFAULT_CMD_PORT  = 3075
-sys.modules['ait'].DEFAULT_CMD_HOST  = '127.0.0.1'
+        return deprecated_func
 
-## Name of the ZMQ topic used to accept commands from external sources
-sys.modules['ait'].DEFAULT_CMD_TOPIC = '__commands__'
+    return deprecated_decorator
 
-## Name of the ZMQ topic / stream used for making telemetry packets available to the script API
-sys.modules['ait'].DEFAULT_TLM_TOPIC = '__tlmpkts__'
 
-## Number of seconds to sleep after ZmqSocket.connect() call, affects clients
-sys.modules['ait'].DEFAULT_CMD_ZMQ_SLEEP = 1
+sys.modules["ait"].deprecated = deprecated  # type: ignore[attr-defined]
+sys.modules["ait"].DEFAULT_CMD_PORT = 3075  # type: ignore[attr-defined]
+sys.modules["ait"].DEFAULT_CMD_HOST = "127.0.0.1"  # type: ignore[attr-defined]
 
+# Name of the ZMQ topic used to accept commands from external sources
+sys.modules["ait"].DEFAULT_CMD_TOPIC = "__commands__"  # type: ignore[attr-defined]
 
+# Name of the ZMQ topic / stream used for making telemetry packets available to the script API
+sys.modules["ait"].DEFAULT_TLM_TOPIC = "__tlmpkts__"  # type: ignore[attr-defined]
 
-sys.modules['ait'].SERVER_DEFAULT_XSUB_URL = "tcp://*:5559"
-sys.modules['ait'].SERVER_DEFAULT_XPUB_URL = "tcp://*:5560"
+# Number of seconds to sleep after ZmqSocket.connect() call, affects clients
+sys.modules["ait"].DEFAULT_CMD_ZMQ_SLEEP = 1  # type: ignore[attr-defined]
+
+
+sys.modules["ait"].SERVER_DEFAULT_XSUB_URL = "tcp://*:5559"  # type: ignore[attr-defined]
+sys.modules["ait"].SERVER_DEFAULT_XPUB_URL = "tcp://*:5560"  # type: ignore[attr-defined]