Merge pull request #867 from ocefpaf/publish_pypi

add PyPI auto-publish

.github/ISSUE_TEMPLATE/compliance-checker-issue-report.md

@@ -9,7 +9,7 @@ assignees: ''
 
 **Version of compliance checker running:**
 
-**Describe the checker this affects:** 
+**Describe the checker this affects:**
 _The ACDD, IOOS, and CF checkers are currently considered in scope for the purposes of the issue tracking on this repository. Issues with other plugins should be placed into their respective repositories._
 
 **Attach a minimal CDL or NetCDF file which is able to reproduce the issue**

.github/workflows/publish.yml

@@ -0,0 +1,46 @@
+name: Publish to PyPI
+
+on:
+  release:
+    types:
+      - published
+
+jobs:
+  packages:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+
+    - name: Set up Python
+      uses: actions/setup-python@v1
+      with:
+        python-version: 3.x
+
+    - name: Get tags
+      run: git fetch --depth=1 origin +refs/tags/*:refs/tags/*
+      shell: bash
+
+    - name: Install build tools
+      run: |
+        python -m pip install --upgrade pip wheel "setuptools>=41.2" setuptools_scm build twine
+      shell: bash
+
+    - name: Build source distribution and binary wheel
+      run: python -m build --sdist --wheel . --outdir dist
+
+    - name: CheckFiles
+      run: |
+        ls dist
+      shell: bash
+
+    - name: Test wheels
+      run: |
+        cd dist && python -m pip install compliance_checker*.whl
+        python -m twine check *
+      shell: bash
+
+    - name: PyPI build
+      uses: pypa/gh-action-pypi-publish@master
+      with:
+        user: __token__
+        password: ${{ secrets.PYPI_ACCESS_TOKEN }}

.pre-commit-config.yaml

@@ -22,7 +22,7 @@ repos:
 #       args: [--max-line-length=200, "--ignore=E203,E501,W503", "--select=select=C,E,F,W,B,B950"]
 
 - repo: https://github.com/pre-commit/mirrors-isort
-  rev: v5.8.0
+  rev: v5.9.3
   hooks:
   - id: isort
     additional_dependencies: [toml]
@@ -34,7 +34,7 @@ repos:
     - id: seed-isort-config
 
 - repo: https://github.com/psf/black
-  rev: 21.6b0
+  rev: 21.7b0
   hooks:
   - id: black
     language_version: python3

compliance_checker/cf/cf.py

@@ -4434,8 +4434,7 @@ def check_variable_features(self, ds):
         # Don't bother checking if it's not a legal featureType
         # if the featureType attribute doesn't exist
         feature_type = getattr(ds, "featureType", "")
-        if (feature_type is not None and
-            feature_type.lower() not in feature_list):
+        if feature_type is not None and feature_type.lower() not in feature_list:
             return []
 
         _feature = feature_type.lower()
@@ -4446,8 +4445,7 @@ def check_variable_features(self, ds):
             if variable_feature is None:
                 continue
             feature_types_found[variable_feature].append(name)
-            matching_feature = TestCtx(BaseCheck.MEDIUM,
-                                       self.section_titles["9.1"])
+            matching_feature = TestCtx(BaseCheck.MEDIUM, self.section_titles["9.1"])
             matching_feature.assert_true(
                 variable_feature.lower() == _feature,
                 "{} is not a {}, it is detected as a {}"

compliance_checker/cfutil.py

@@ -2010,7 +2010,6 @@ def guess_feature_type(nc, variable):
         return "reduced-grid"
 
 
-
 def units_convertible(units1, units2, reftimeistime=True):
     """
     Return True if a Unit representing the string units1 can be converted