Merge pull request #53 from ocefpaf/GHA

Move to GitHub Actions

Author: ocefpaf

.github/dependabot.yml

@@ -0,0 +1,11 @@
+# See https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/keeping-your-actions-up-to-date-with-dependabot
+
+version: 2
+updates:
+
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "daily"
+    labels:
+      - "Bot"

.github/workflows/tests.yml

@@ -0,0 +1,37 @@
+name: Default Tests
+
+on:
+  pull_request:
+  push:
+
+jobs:
+  run:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
+        os: [windows-latest, ubuntu-latest, macos-latest]
+      fail-fast: false
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Setup Micromamba ${{ matrix.python-version }}
+      uses: mamba-org/setup-micromamba@v1
+      with:
+        environment-name: TEST
+        init-shell: bash
+        create-args: >-
+          python=${{ matrix.python-version }} pip
+          --file requirements.txt
+          --file requirements-dev.txt
+          --channel conda-forge
+
+    - name: Install compliance-checker
+      shell: bash -l {0}
+      run: |
+        python -m pip install -e . --no-deps --force-reinstall
+
+    - name: Default Tests
+      shell: bash -l {0}
+      run: pytest -s -rxs -v cc_plugin_ncei

.travis.yml

@@ -1,21 +1,22 @@
-from pkg_resources import resource_filename
-import sh
+from pathlib import Path
+import subprocess
 import os
 
 
 def get_filename(path):
     '''
     Returns the path to a valid dataset
     '''
-    filename = resource_filename('cc_plugin_ncei', path)
+    resource_filename = Path(__file__).parent.parent
+    filename = resource_filename.joinpath(path)
     if not os.path.exists(filename):
-        cdl_path = filename.replace('.nc', '.cdl')
+        cdl_path = filename.with_suffix('.cdl')
         generate_dataset(cdl_path, filename)
     return filename
 
 
 def generate_dataset(cdl_path, nc_path):
-    sh.ncgen('-o', nc_path, cdl_path)
+    subprocess.run(["ncgen", '-o', nc_path, cdl_path])
 
 
 STATIC_FILES = {

cc_plugin_ncei/tests/resources.py

@@ -1,9 +1,7 @@
-#!/usr/bin/env python
-# -*- coding: utf-8 -*-
 '''
 cc_plugin_ncei/util.py
 '''
-from pkg_resources import resource_filename
+from pathlib import Path
 import json
 from lxml import etree
 from pkgutil import get_data
@@ -18,9 +16,10 @@ def get_unitless_standard_names():
     Returns a list of valid standard_names that are allowed to be unitless
     '''
     global _UNITLESS_DB
+    resource_filename = Path(__file__).parent
     if _UNITLESS_DB is None:
-        with open(resource_filename('cc_plugin_ncei', 'data/unitless.json'), 'r') as f:
-            _UNITLESS_DB = json.load(f)
+        f = resource_filename.joinpath('data/unitless.json').read_text()
+        _UNITLESS_DB = json.loads(f)
     return _UNITLESS_DB
 
 

cc_plugin_ncei/util.py

@@ -4,5 +4,4 @@ flake8-comprehensions
 flake8-mutable
 flake8-print
 pytest
-pytest-flake8
-sh
+pytest-flake8