Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

global: migrate CI to GH Actions #94

Merged
merged 1 commit into from
Dec 2, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Major: Same as inveniosoftware/invenio-oauth2server#220 (comment), missing docker-services-cli.

check-manifest --ignore ".*-requirements.txt"
sphinx-build -qnNW docs docs/_build/html
docker-services-cli up ${DB}
python setup.py test
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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