Skip to content

Commit

Permalink
Add github action to for testing pytest; remove travis
Browse files Browse the repository at this point in the history
- Test (with pytest) using Github Actions, since Travis is now
  limiting free projects.  See
  jupyterhub/team-compass#353 .
- Almost a direct translation of current .travis.yml, only minor edits
  to bring it up to date with some versions.
- Remove .travis.yml
- Review:
  - I don't know if codecov is used for anything
  - Is everyone happy with the python/JH version coverage?
  - General check for best practices
  • Loading branch information
rkdarst committed Dec 2, 2020
1 parent 33b4a7b commit ab788dd
Show file tree
Hide file tree
Showing 2 changed files with 51 additions and 28 deletions.
51 changes: 51 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@

name: Python package

on: [push, pull_request]

jobs:
build:

runs-on: ubuntu-20.04
continue-on-error: ${{ matrix.allow_failure }}
strategy:
matrix:
python-version:
- '3.5'
- '3.9'
JHUB_VER:
- '0.9.6'
- '1.0.0'
- '1.1.0'
- '1.2.0'
allow_failure: [false]
include:
- JHUB_VER: 'master'
python-version: '3.9'
allow_failure: true

steps:
- uses: actions/checkout@v2
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}

- name: Install dependencies
run: |
python -m pip install --upgrade pip
python -m pip install pytest
pip install -r requirements.txt
- name: Install nodejs dependencies
run: |
sudo npm install -g configurable-http-proxy
- name: install JupyterHub
run: |
git clone --quiet --branch ${{ matrix.JHUB_VER }} https://github.com/jupyterhub/jupyterhub.git jupyterhub
pip install --pre -r jupyterhub/dev-requirements.txt
pip install --upgrade pytest
pip install --pre -e jupyterhub
- name: pytest
run: |
pytest --lf --cov batchspawner batchspawner/tests -v
28 changes: 0 additions & 28 deletions .travis.yml

This file was deleted.

0 comments on commit ab788dd

Please sign in to comment.