fix: add initial CI, including semantic release

This also fixes any pre-commit issues

Signed-off-by: Patrick Avery <[email protected]>

Author: psavery

.codespellrc

@@ -0,0 +1,3 @@
+[codespell]
+skip = **/package-lock.json
+ignore-words-list = doubleclick

.flake8

@@ -0,0 +1,9 @@
+[flake8]
+
+# Just assume black did a good job with the line lengths
+ignore =
+    E501
+
+per-file-ignores =
+    # These directories will always contain "from ... import *"
+    trame/*:F401,F403

.github/workflows/test_and_release.yml

@@ -0,0 +1,110 @@
+name: Test and Release
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: actions/setup-python@v2
+        with:
+          python-version: "3.9"
+
+      # Install and run pre-commit
+      - run: |
+          pip install pre-commit
+          pre-commit install
+          pre-commit run --all-files
+
+  pytest:
+    name: Pytest ${{ matrix.config.name }}
+    runs-on: ${{ matrix.config.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: [3.9]
+        config:
+        - {
+            name: "Linux",
+            os: ubuntu-latest
+          }
+        - {
+            name: "MacOSX",
+            os: macos-latest
+          }
+        - {
+            name: "Windows",
+            os: windows-latest
+          }
+
+    defaults:
+      run:
+        shell: bash
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v2
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install and Run Tests
+      run: |
+        pip install .
+        pip install -r tests/requirements.txt
+        pytest -s ./tests
+
+  test-npm-build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Set Up Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 14
+
+      - name: Build Vue Components
+        run: |
+          cd vue-components
+          npm ci
+          npm run build
+
+  release:
+    needs: [pre-commit, pytest, test-npm-build]
+    runs-on: ubuntu-latest
+    if: github.event_name == 'push'
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+
+      - name: Set Up Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 14
+
+      - name: Build Vue Components
+        run: |
+          cd vue-components
+          npm ci
+          npm run build
+
+      - name: Python Semantic Release
+        uses: relekang/[email protected]
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          repository_username: __token__
+          repository_password: ${{ secrets.PYPI_API_TOKEN }}

.pre-commit-config.yaml

@@ -0,0 +1,16 @@
+repos:
+  - repo: https://github.com/psf/black
+    rev: 22.3.0
+    hooks:
+      - id: black
+        entry: black --check
+
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.1.0
+    hooks:
+      - id: codespell
+
+  - repo: https://github.com/PyCQA/flake8
+    rev: 4.0.1
+    hooks:
+      - id: flake8

README.rst

@@ -6,7 +6,7 @@ trame-plotly extend trame **widgets** with components that can interface with Pl
 Plotly integration in trame allow you to create rich visualization by leveraging their `Python <https://plotly.com/python/>`_ or `JavaScript <https://plotly.com/javascript/>`_ interface.
 The JavaScript version is exposed via `Vue.plotly <https://david-desmaisons.github.io/vue-plotly/>`_ within trame.widgets.plotly.Plotly class definition.
 
-This package is not supposed to be used by iteself but rather should come as a dependency of **trame**.
+This package is not supposed to be used by itself but rather should come as a dependency of **trame**.
 For any specificity, please refer to `the trame documentation <https://kitware.github.io/trame/>`_.
 
 

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = trame-plotly
-version = 2.0.0rc6
+version = 2.0.0
 description = Plotly figure widget for trame
 long_description = file: README.rst
 long_description_content_type = text/x-rst
@@ -28,3 +28,6 @@ packages = find:
 include_package_data = True
 install_requires =
     trame-client
+
+[semantic_release]
+version_pattern = setup.cfg:version = (\d+\.\d+\.\d+)

tests/requirements.txt

@@ -0,0 +1 @@
+pytest

tests/test_import.py

@@ -0,0 +1,2 @@
+def test_import():
+    from trame.widgets.plotly import Figure  # noqa: F401

trame_plotly/widgets/plotly.py

@@ -65,6 +65,7 @@ class Figure(AbstractElement):
     :param transition_interrupted:
     :param unhover:
     """
+
     _next_id = 0
 
     def __init__(self, figure=None, **kwargs):