Skip to content

Commit

Permalink
Merge pull request #28 from ekwan/development
Browse files Browse the repository at this point in the history
Merge CI and Dependency Fixes
  • Loading branch information
jlw387 authored Aug 23, 2024
2 parents f2047cc + 9ce1c2f commit 29a5aaa
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 17 deletions.
48 changes: 34 additions & 14 deletions .github/workflows/python-app.yml
Original file line number Diff line number Diff line change
@@ -1,40 +1,60 @@
# This workflow will install Python dependencies, run tests and lint with a single version of Python
# This workflow will install Python dependencies, run tests and lint with a variety of Python versions
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python

name: Python application
name: Python package

on:
push:
branches: [ "main" ]
branches: [ "main", "development" ]
pull_request:
branches: [ "main" ]

permissions:
contents: read
branches: [ "main", "development" ]

jobs:
run-tests:
build:

runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
python-version: ["3.10", "3.11", "3.12"]

steps:
- uses: actions/checkout@v4
- name: Set up Python 3.10
uses: actions/setup-python@v3

- name: Check branch names
id: check_branch_names
run: |
BASE_REF="${{ github.base_ref }}"
HEAD_REF="${{ github.head_ref }}"
echo "Base ref: $BASE_REF"
echo "Head ref: $HEAD_REF"
if [[ "$BASE_REF" == "main" && ! (“$HEAD_REF == “development || "$HEAD_REF" =~ ^hotfix_) ]]; then
echo "Main cannot be merged from branches other than development or hotfixes. Exiting..."
exit 1
fi
echo "Branches are merge compatible."
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
with:
python-version: "3.10"
python-version: ${{ matrix.python-version }}

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install flake8 pytest
python -m pip install flake8 pytest
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Lint with flake8
run: |
# stop the build if there are Python syntax errors or undefined names
flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
flake8 pyplate --count --exit-zero --max-line-length=127 --statistics
flake8 tests --count --exit-zero --max-line-length=127 --statistics
flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
- name: Test with pytest
run: |
pytest tests
6 changes: 3 additions & 3 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
numpy==1.23.5
numpy>=1.23.5
pytest>=7.1.2
pytest-cov>=4.1.0
pyyaml>=6.0.1
pandas==1.5.3
tabulate==0.9.0
pandas>=1.5.3
tabulate>=0.9.0

0 comments on commit 29a5aaa

Please sign in to comment.