Skip to content

run tests on all platforms #4

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.

Sign up for GitHub

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

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

run tests on all platforms #4

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
32e5a96
remove xml support
kd7lxl Aug 23, 2021
4becd24
run tests on all platforms
kd7lxl Aug 23, 2021
3e046e1
use python 3 on py3 branch
kd7lxl Aug 23, 2021
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
21 changes: 11 additions & 10 deletions .github/workflows/test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,21 @@ on:
jobs:
test:
name: Run all tests
runs-on: ubuntu-latest
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
steps:
- uses: actions/checkout@v2
- name: install dependencies
- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: ${{ github.event.pull_request.base.ref == 'py3' && '3.x' || '2.7' }}
- name: Install dependencies
run: |
sudo apt-get remove -y python3-libxml2
sudo apt-get install -y libxml2-dev python-dev python-libxml2
curl https://bootstrap.pypa.io/pip/2.7/get-pip.py --output get-pip.py
python get-pip.py
pip --version
python -m pip install --upgrade --no-deps --force-reinstall lxml
python -m pip install \
future \
pep8 \
pyserial \
tox
- run: rm chirp/drivers/generic_xml.py
Copy link
Owner

@kk7ds kk7ds Aug 23, 2021

Choose a reason for hiding this comment

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

I would rather just remove it from the tree than have the test stuff silently behave differently here versus locally. I have a commit on the py3 branch to do it, I'll pull it over.

- run: tox