Add ci for py39 (#44)

* add ci for 39

* add gh actions

* black

Author: almarklein

.github/workflows/ci.yml

@@ -0,0 +1,85 @@
+name: CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+
+jobs:
+  build:
+    name: ${{ matrix.name }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        include:
+          - name: Lint
+            os: ubuntu-latest
+            pyversion: '3.7'
+            dolint: 1
+          - name: Linux py34
+            os: ubuntu-latest
+            pyversion: '3.6'
+            tests: 1
+          - name: Linux py36
+            os: ubuntu-latest
+            pyversion: '3.6'
+            tests: 1
+          - name: Linux py37
+            os: ubuntu-latest
+            pyversion: '3.7'
+            tests: 1
+          - name: Linux py38
+            os: ubuntu-latest
+            pyversion: '3.8'
+            tests: 1
+          - name: Linux py39
+            os: ubuntu-latest
+            pyversion: '3.9'
+            tests: 1
+          - name: Linux pypy3
+            os: ubuntu-latest
+            pyversion: 'pypy3'
+            tests: 1
+          - name: Windows py38
+            os: windows-latest
+            pyversion: '3.8'
+            tests: 1
+          - name: MacOS py38
+            os: macos-latest
+            pyversion: '3.8'
+            tests: 1
+
+    steps:
+
+      - uses: actions/checkout@v2
+      - name: Set up Python ${{ matrix.pyversion }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.pyversion }}
+
+      - name: Install dependencies (lint and docs)
+        if: matrix.dolint == 1
+        run: |
+            python -m pip install --upgrade pip
+            pip install invoke black flake8
+      - name: Install dependencies (unit tests)
+        if: matrix.tests == 1
+        run: |
+            python -m pip install --upgrade pip
+            pip install psutil
+            pip install invoke pytest pytest-cov
+            invoke get-ffmpeg-binary;
+
+      - name: Lint
+        if: matrix.dolint == 1
+        run: |
+            invoke lint
+            invoke checkformat
+      - name: Test with pytest
+        if: matrix.tests == 1
+        run: |
+            python -c "import sys; print(sys.version, '\n', sys.prefix)";
+            python -c 'import imageio_ffmpeg; print(imageio_ffmpeg.get_ffmpeg_version())'
+            invoke test

.travis.yml

@@ -40,7 +40,10 @@ matrix:
       python: "3.7"
       env: TEST_UNIT=1
     - os: linux
-      python: "3.8-dev"
+      python: "3.8"
+      env: TEST_UNIT=1
+    - os: linux
+      python: "3.9"
       env: TEST_UNIT=1
 
 

imageio_ffmpeg/_parsing.py

@@ -6,7 +6,7 @@
 
 
 class LogCatcher(threading.Thread):
-    """ Thread to keep reading from stderr so that the buffer does not
+    """Thread to keep reading from stderr so that the buffer does not
     fill up and stalls the ffmpeg process. On stderr a message is send
     on every few frames with some meta information. We only keep the
     last ones.
@@ -27,12 +27,11 @@ def stop_me(self):
 
     @property
     def header(self):
-        """ Get header text. Empty string if the header is not yet parsed.
-        """
+        """Get header text. Empty string if the header is not yet parsed."""
         return self._header
 
     def get_text(self, timeout=0):
-        """ Get the whole text written to stderr so far. To preserve
+        """Get the whole text written to stderr so far. To preserve
         memory, only the last 50 to 100 frames are kept.
 
         If a timeout is given, wait for this thread to finish. When
@@ -93,15 +92,14 @@ def get_output_video_line(lines):
 
 
 def limit_lines(lines, N=32):
-    """ When number of lines > 2*N, reduce to N.
-    """
+    """When number of lines > 2*N, reduce to N."""
     if len(lines) > 2 * N:
         lines = [b"... showing only last few lines ..."] + lines[-N:]
     return lines
 
 
 def cvsecs(*args):
-    """ converts a time to second. Either cvsecs(min, secs) or
+    """converts a time to second. Either cvsecs(min, secs) or
     cvsecs(hours, mins, secs).
     """
     if len(args) == 1:

tasks.py

@@ -25,8 +25,7 @@
 
 @task
 def test(ctx, cover=False):
-    """Perform unit tests. Use --cover to open a webbrowser to show coverage.
-    """
+    """Perform unit tests. Use --cover to open a webbrowser to show coverage."""
     cmd = [sys.executable, "-m", "pytest", "tests", "-v"]
     cmd += ["--cov=" + LIBNAME, "--cov-report=term", "--cov-report=html"]
     ret_code = subprocess.call(cmd, cwd=ROOT_DIR)
@@ -40,8 +39,7 @@ def test(ctx, cover=False):
 
 @task
 def lint(ctx):
-    """ Validate the code style (e.g. undefined names)
-    """
+    """Validate the code style (e.g. undefined names)"""
     try:
         importlib.import_module("flake8")
     except ImportError:
@@ -61,24 +59,21 @@ def lint(ctx):
 
 @task
 def checkformat(ctx):
-    """ Check whether the code adheres to the style rules. Use autoformat to fix.
-    """
+    """Check whether the code adheres to the style rules. Use autoformat to fix."""
     print("Checking format with black (also see invoke autoformat):")
     black_wrapper(False)
 
 
 @task
 def autoformat(ctx):
-    """ Automatically format the code (using black).
-    """
+    """Automatically format the code (using black)."""
     print("Auto-formatting with black:")
     black_wrapper(True)
 
 
 @task
 def clean(ctx):
-    """ Clean the repo of temp files etc.
-    """
+    """Clean the repo of temp files etc."""
     for root, dirs, files in os.walk(ROOT_DIR):
         for dname in dirs:
             if dname in (
@@ -104,8 +99,7 @@ def clean(ctx):
 
 @task
 def get_ffmpeg_binary(ctx):
-    """ Download/copy ffmpeg binary for local development.
-    """
+    """Download/copy ffmpeg binary for local development."""
     # Get ffmpeg fname
     sys.path.insert(0, os.path.join(ROOT_DIR, "imageio_ffmpeg"))
     try:
@@ -139,8 +133,7 @@ def get_ffmpeg_binary(ctx):
 
 @task
 def build(ctx):
-    """ Build packages for different platforms. Dont release yet.
-    """
+    """Build packages for different platforms. Dont release yet."""
 
     # Get version and more
     sys.path.insert(0, os.path.join(ROOT_DIR, "imageio_ffmpeg"))
@@ -213,8 +206,7 @@ def build(ctx):
 
 @task
 def release(ctx):
-    """ Release the packages to Pypi!
-    """
+    """Release the packages to Pypi!"""
     dist_dir = os.path.join(ROOT_DIR, "dist")
     if not os.path.isdir(dist_dir):
         sys.exit("Dist directory does not exist. Build first?")
@@ -239,8 +231,7 @@ def release(ctx):
 
 @task
 def update_readme(ctx):
-    """Update readme to include the latest API docs.
-    """
+    """Update readme to include the latest API docs."""
     text = open(os.path.join(ROOT_DIR, "README.md"), "rb").read().decode()
     text = text.split("\n## API\n")[0] + "\n## API\n\n"
 
@@ -267,8 +258,7 @@ def update_readme(ctx):
 
 
 def black_wrapper(writeback):
-    """ Helper function to invoke black programatically.
-    """
+    """Helper function to invoke black programatically."""
 
     check = [] if writeback else ["--check"]
     exclude = "|".join(["cangivefilenameshere"])