Skip to content

Commit

Permalink
Clean up GitHub Actions workflows (#883)
Browse files Browse the repository at this point in the history
Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
  • Loading branch information
mwtoews and AA-Turner authored Jul 30, 2024
1 parent 6c86259 commit e1b995e
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 38 deletions.
14 changes: 5 additions & 9 deletions .github/workflows/documentation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,20 +6,16 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2
- uses: actions/cache@v1
with:
path: ~/.cache/pip
key: ${{ runner.os }}-pip-${{ hashFiles('requirements/*.txt') }}
restore-keys: |
${{ runner.os }}-pip-
- uses: actions/checkout@v4

- name: set up python
uses: actions/setup-python@v1
uses: actions/setup-python@v5
with:
cache: 'pip'

- name: install dependencies
run: |
pip install ".[docs,lint,test]"
pip install ".[docs]"
sudo apt-get -y update
sudo apt-get -y install graphviz libclang1-11 libclang-cpp11
Expand Down
18 changes: 6 additions & 12 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,19 +6,16 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2
- uses: actions/cache@v1
with:
path: ~/.cache/pip
key: ${{ runner.os }}-pip-${{ hashFiles('requirements/*.txt') }}
restore-keys: |
${{ runner.os }}-pip-
- uses: actions/checkout@v4

- name: set up python
uses: actions/setup-python@v1
uses: actions/setup-python@v5
with:
cache: 'pip'

- name: install dependencies
run: |
pip install ".[docs,lint,test]"
pip install ".[lint]"
- name: lint the source code
run: make flake8
Expand All @@ -28,6 +25,3 @@ jobs:

- name: type check the source code
run: make type-check

- name: check version numbers are in sync
run: make version-check
13 changes: 4 additions & 9 deletions .github/workflows/unit_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,17 +36,12 @@ jobs:
sphinx-version: '6.1.3'

steps:
- uses: actions/checkout@v2
- uses: actions/cache@v1
with:
path: ~/.cache/pip
key: ${{ runner.os }}-pip-${{ hashFiles('requirements/*.txt') }}
restore-keys: |
${{ runner.os }}-pip-
- uses: actions/checkout@v4
- name: set up python ${{ matrix.python-version }}
uses: actions/setup-python@v1
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: 'pip'

- name: install sphinx from PyPI or from git
run: |
Expand All @@ -58,7 +53,7 @@ jobs:
- name: install dependencies
run: |
pip install .[
pip install ".[test]"
- name: run the unit tests
run: make dev-test
4 changes: 0 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,3 @@ black:
.PHONY: type-check
type-check:
mypy --warn-redundant-casts --warn-unused-ignores breathe tests

.PHONY: version-check
version-check:
PYTHONPATH=../:$(PYTHONPATH) python3 scripts/version-check.py
1 change: 0 additions & 1 deletion breathe-apidoc.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#!/usr/bin/env python
# -*- coding: utf-8 -*-

import sys

Expand Down
2 changes: 0 additions & 2 deletions documentation/source/conf.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
# -*- coding: utf-8 -*-
#
# BreatheExample documentation build configuration file, created by
# sphinx-quickstart on Tue Feb 3 18:20:48 2009.
#
Expand Down
1 change: 1 addition & 0 deletions mkrelease
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ upload()
(
cd -- mkrelease_tmp

twine check --strict dist/*
twine upload -- dist/*

{
Expand Down
1 change: 0 additions & 1 deletion tests/warnings/source/conf.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
#
# Test Breathe Warnings documentation build configuration file, created by
# sphinx-quickstart on Thu Jun 5 18:57:21 2014.
#
Expand Down

0 comments on commit e1b995e

Please sign in to comment.