CI improvements (lyst#541)

* move circle.yml to the default location .circleci/config.yml

* added 2.0/2.1 workflows

* allow restore_cache to restore from multiple keys

* expose test results to CircleCI

* test using multiple python versions

* ignoring non-pep8 compliant E203 error

* fix appveyor reference to requirements

* black-ify setup.py and remove deprecated test command

* update travis osx build to py3

Author: Igor Serko

.circleci/config.yml

@@ -0,0 +1,82 @@
+version: 2.1
+
+jobs:
+  pytest:
+    parameters:
+      image:
+        description: "Docker image to run tests in"
+        default: "circleci/python:3.8.3-buster-browsers"
+        type: string
+      python_version:
+        description: "Python version used to separate cache"
+        default: "py38"
+        type: string
+    docker:
+      - image: << parameters.image >>
+    steps:
+      - checkout
+      - restore_cache:
+          keys:
+            - deps-<< parameters.python_version>>-{{ .Branch }}-{{ checksum "test-requirements.txt" }}
+            - deps-<< parameters.python_version>>-{{ .Branch }}-
+            - deps-<< parameters.python_version>>-
+      - run:
+          name: Install test requirements
+          command: |
+            python3 -m venv venv
+            . venv/bin/activate
+            pip install -r test-requirements.txt .
+      - save_cache:
+          key: deps-<< parameters.python_version>>-{{ .Branch }}-{{ checksum "test-requirements.txt" }}
+          paths:
+            - "venv"
+      - run:
+          name: Test run
+          command: |
+            . venv/bin/activate
+            mkdir test_results
+            mv lightfm _lightfm
+            py.test -xv --junitxml=test_results/junit.xml tests
+      - store_test_results:
+          path: test_results
+  lint:
+    docker:
+      - image: circleci/python:3.8.3-buster
+    steps:
+      - checkout
+      - restore_cache:
+          keys:
+            - deps-lint-{{ .Branch }}-{{ checksum "lint-requirements.txt"}}
+            - deps-lint-{{ .Branch }}-
+            - deps-lint-
+      - run:
+          name: Install lint requirements
+          command: |
+            python3 -m venv venv
+            . venv/bin/activate
+            pip install -r "lint-requirements.txt"
+      - save_cache:
+          key: deps-lint-{{ .Branch }}-{{ checksum "lint-requirements.txt" }}
+          paths:
+            - "venv"
+      - run:
+          name: Lint
+          command: |
+            . venv/bin/activate
+            flake8 setup.py lightfm tests
+            black --check setup.py lightfm tests
+
+workflows:
+  version: 2
+  test:
+    jobs:
+      - lint
+      - pytest:
+          image: circleci/python:3.6.10-stretch-browsers
+          python_version: py36
+      - pytest:
+          image: circleci/python:3.7.7-buster-browsers
+          python_version: py37
+      - pytest:
+          image: circleci/python:3.8.3-buster-browsers
+          python_version: py38

.gitignore

@@ -1,3 +1,4 @@
+venv/
 *.pyc
 *.egg*
 *~

.travis.yml

@@ -1,7 +1,8 @@
 os:
   - osx
 before_install:
-  - pip2 install -r dev-requirements.txt
+  - python3 -m venv venv
+  - venv/bin/pip install -r test-requirements.txt
 install:
-  - pip2 install -e .
-script: py.test tests/
+  - venv/bin/pip install -e .
+script: venv/bin/py.test -xv tests/

appveyor.yml

@@ -17,7 +17,7 @@ install:
   - activate lightfm
 
   # Install the build dependencies of the project.
-  - conda install --yes --file dev-requirements.txt
+  - conda install --yes --file test-requirements.txt
 
   # Install lightfm
   - pip install -e .

circle.yml

@@ -0,0 +1,2 @@
+black
+flake8

dev-requirements.txt

@@ -2,7 +2,7 @@
 description-file = README.md
 
 [flake8]
-ignore = I100, W503
+ignore = I100, W503, E203
 max-line-length = 100
 exclude = .git,__pycache__,docs/source/conf.py,old,build,dist,docs,doc