Skip to content

Commit

Permalink
global: migrate CI to GH Actions
Browse files Browse the repository at this point in the history
closes #93
  • Loading branch information
Parth Shandilya authored and Diego committed Dec 2, 2020
1 parent 07b2449 commit d697049
Show file tree
Hide file tree
Showing 10 changed files with 133 additions and 88 deletions.
4 changes: 2 additions & 2 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@ indent_size = 4
[*.{css,html,js,json,yml}]
indent_size = 2

# Matches the exact files either package.json or .travis.yml
[{package.json,.travis.yml}]
# Matches the exact files either package.json or .github/workflows/*.yml
[{package.json,.github/workflows/*.yml}]
indent_size = 2

# Dockerfile
Expand Down
37 changes: 37 additions & 0 deletions .github/workflows/pypi-release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
name: Publish

on:
push:
tags:
- v*

jobs:
Publish:

runs-on: ubuntu-20.04

steps:
- name: Checkout
uses: actions/checkout@v2

- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: 3.7

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install setuptools wheel
- name: Build package
# Remove `compile_catalog` if the package has no translations.
run: |
python setup.py compile_catalog sdist bdist_wheel
- name: Publish on PyPI
uses: pypa/[email protected]
with:
user: __token__
# The token is provided by the inveniosoftware organization
password: ${{ secrets.pypi_token }}
72 changes: 72 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
name: CI

on:
push:
branches: master
pull_request:
branches: master
schedule:
# * is a special character in YAML so you have to quote this string
- cron: '0 3 * * 6'
workflow_dispatch:
inputs:
reason:
description: 'Reason'
required: false
default: 'Manual trigger'

jobs:
Tests:
runs-on: ubuntu-20.04
strategy:
matrix:
python-version: [3.6, 3.7, 3.8]
requirements-level: [min, pypi]
db-service: [postgresql9]

exclude:
- python-version: 3.8
requirements-level: min

include:
- db-service: postgresql9
DB: postgresql
POSTGRESQL_VERSION: POSTGRESQL_9_LATEST
SQLALCHEMY_DATABASE_URI: "postgresql+psycopg2://invenio:invenio@localhost:5432/invenio"
EXTRAS: "all,postgresql"

env:
SQLALCHEMY_DATABASE_URI: ${{matrix.SQLALCHEMY_DATABASE_URI}}
POSTGRESQL_VERSION: ${{matrix.POSTGRESQL_VERSION}}
DB: ${{ matrix.DB }}

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: Generate dependencies
run: |
pip uninstall -y six
python -m pip install --upgrade pip setuptools py wheel requirements-builder
requirements-builder -e ${{ matrix.EXTRAS }} ${{ matrix.requirements-file }} --level=${{ matrix.requirements-level }} setup.py > .${{ matrix.requirements-level }}-${{ matrix.python-version }}-requirements.txt
- name: Cache pip
uses: actions/cache@v2
with:
path: ~/.cache/pip
key: ${{ runner.os }}-pip-${{ hashFiles('.${{ matrix.requirements-level }}-${{ matrix.python-version }}-requirements.txt') }}

- name: Install dependencies
run: |
pip install -r .${{matrix.requirements-level}}-${{ matrix.python-version }}-requirements.txt
pip install .[${{ matrix.EXTRAS }}]
pip freeze
- name: Run tests
run: |
./run-tests.sh
69 changes: 0 additions & 69 deletions .travis.yml

This file was deleted.

4 changes: 2 additions & 2 deletions CONTRIBUTING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,6 @@ Before you submit a pull request, check that it meets these guidelines:
1. The pull request should include tests and must not decrease test coverage.
2. If the pull request adds functionality, the docs should be updated. Put
your new functionality into a function with a docstring.
3. The pull request should work for Python 2.7, 3.5 and 3.6. Check
https://travis-ci.org/inveniosoftware/invenio-records-ui/pull_requests
3. The pull request should work for Python 3.6, 3.7 and 3.8. Check
https://github.com/inveniosoftware/invenio-records-ui/actions?query=event%3Apull_request
and make sure that the tests pass for all supported Python versions.
1 change: 1 addition & 0 deletions MANIFEST.in
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ recursive-include docs Makefile
recursive-include examples *.py
recursive-include examples *.sh
recursive-include examples *.txt
recursive-include .github/workflows *.yml
recursive-include invenio_records_ui *.html
recursive-include invenio_records_ui *.po *.pot *.mo
recursive-include tests *.py
4 changes: 2 additions & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
.. image:: https://img.shields.io/github/license/inveniosoftware/invenio-records-ui.svg
:target: https://github.com/inveniosoftware/invenio-records-ui/blob/master/LICENSE

.. image:: https://img.shields.io/travis/inveniosoftware/invenio-records-ui.svg
:target: https://travis-ci.org/inveniosoftware/invenio-records-ui
.. image:: https://github.com/inveniosoftware/invenio-records-ui/workflows/CI/badge.svg
:target: https://github.com/inveniosoftware/invenio-records-ui/actions

.. image:: https://img.shields.io/coveralls/inveniosoftware/invenio-records-ui.svg
:target: https://coveralls.io/r/inveniosoftware/invenio-records-ui
Expand Down
2 changes: 1 addition & 1 deletion pytest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,6 @@

[pytest]
pep8ignore = docs/conf.py ALL
addopts = --pep8 --doctest-glob="*.rst" --doctest-modules --cov=invenio_records_ui --cov-report=term-missing
addopts = --isort --pydocstyle --pycodestyle --doctest-glob="*.rst" --doctest-modules --cov=invenio_records_ui --cov-report=term-missing tests invenio_records_ui
testpaths = docs tests invenio_records_ui
filterwarnings = ignore::pytest.PytestDeprecationWarning
18 changes: 14 additions & 4 deletions run-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,18 @@
# Invenio is free software; you can redistribute it and/or modify it
# under the terms of the MIT License; see LICENSE file for more details.

pydocstyle invenio_records_ui tests docs && \
isort invenio_records_ui tests --check-only --diff && \
check-manifest --ignore ".travis-*" && \
sphinx-build -qnNW docs docs/_build/html && \
# Quit on errors
set -o errexit

# Quit on unbound symbols
set -o nounset

pydocstyle invenio_records_ui tests docs
isort invenio_records_ui tests --check-only --diff
check-manifest --ignore ".*-requirements.txt"
sphinx-build -qnNW docs docs/_build/html
docker-services-cli up ${DB}
python setup.py test
tests_exit_code=$?
docker-services-cli down
exit "$tests_exit_code"
10 changes: 2 additions & 8 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,10 @@
history = open('CHANGES.rst').read()

tests_require = [
'check-manifest>=0.25',
'coverage>=4.0',
'invenio-access>=1.0.0',
'invenio-accounts>=1.2.0',
'invenio-accounts>=1.3.0',
'invenio-db[versioning,mysql,postgresql]>=1.0.0',
'isort>=4.2.2',
'pydocstyle>=1.0.0',
'pytest-cov>=1.8.0',
'pytest-pep8>=1.0.6',
'pytest>=2.8.0',
'pytest-invenio>=1.4.0',
]

extras_require = {
Expand Down

0 comments on commit d697049

Please sign in to comment.