Skip to content

Commit

Permalink
[qa] Updated CI, caching of pip deps, minor QA improvements
Browse files Browse the repository at this point in the history
  • Loading branch information
nemesifier committed Aug 5, 2024
1 parent c7d1911 commit f0cdb42
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 6 deletions.
16 changes: 12 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ on:
jobs:
qa-checks:
name: QA-Checks
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04

steps:
- uses: actions/checkout@v4
Expand All @@ -35,18 +35,20 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.10"
cache: 'pip'
cache-dependency-path: **/requirements*.txt

- name: Installing dependencies
run: |
yarn install
pip install openwisp-utils[qa]
pip install -r requirements-qa.txt
- name: QA checks
run: ./run-qa-checks

build:
name: Tests and Coverage
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04

steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -75,6 +77,10 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.10"
with:
python-version: "3.10"
cache: 'pip'
cache-dependency-path: **/requirements*.txt

- name: Get openwisp-radius
run: |
Expand All @@ -90,7 +96,9 @@ jobs:

- name: Installing OpenWISP Radius
run: |
cd openwisp-radius && pip install -U -e ".[saml]" "Django~=4.2.0"
cd openwisp-radius
pip install -U -e .
pip install -U -r requirements-test.txt
./tests/manage.py migrate
- name: Creating configuration of organization (supports mobile verification)
Expand Down
3 changes: 1 addition & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ OpenWISP documentation for a complete overview of features.
Documentation
-------------

- `Usage documentation
<https://openwisp.io/docs/dev/wifi-login-pages/>`_
- `Usage documentation <https://openwisp.io/docs/dev/wifi-login-pages/>`_
- `Developer documentation
<https://openwisp.io/docs/dev/wifi-login-pages/developer/index.html>`_

Expand Down
1 change: 1 addition & 0 deletions requirements-qa.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
openwisp-utils[qa] @ https://github.com/openwisp/openwisp-utils/tarball/master
1 change: 1 addition & 0 deletions requirements-tests.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
openwisp-utils[qa] @ https://github.com/openwisp/openwisp-utils/tarball/master

0 comments on commit f0cdb42

Please sign in to comment.