Skip to content

Commit 6bde0c2

Browse files
authored
Merge pull request #1533 from ocefpaf/test_split
Test split
2 parents cc8fb87 + a3f6023 commit 6bde0c2

4 files changed

Lines changed: 54 additions & 36 deletions

File tree

.github/workflows/publish.yml

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
11
name: Publish to PyPI
22

33
on:
4+
push:
5+
branches:
6+
- main
7+
pull_request:
8+
branches:
9+
- main
410
release:
511
types:
612
- published
@@ -14,40 +20,34 @@ jobs:
1420
- name: Set up Python
1521
uses: actions/setup-python@v1
1622
with:
17-
python-version: 3.8
23+
python-version: 3.x
1824

1925
- name: Get tags
2026
run: git fetch --depth=1 origin +refs/tags/*:refs/tags/*
2127
shell: bash
2228

2329
- name: Install build tools
2430
run: |
25-
python -m pip install --upgrade pip wheel setuptools setuptools_scm twine "branca>=0.3.0" "jinja2>=2.9" numpy requests
31+
python -m pip install --upgrade pip wheel build twine
2632
2733
shell: bash
2834

29-
- name: Build binary wheel
30-
run: pip wheel . -w dist --no-deps
31-
32-
- name: Build sdist
33-
run: python setup.py sdist
35+
- name: Build binary wheel and sdist
36+
run: python -m build --sdist --wheel . --outdir dist
3437

3538
- name: CheckFiles
3639
run: |
3740
ls dist
3841
shell: bash
3942

40-
- name: Check Version
41-
run: |
42-
python setup.py --version
43-
4443
- name: Test wheels
4544
run: |
4645
cd dist && python -m pip install folium*.whl
4746
python -m twine check *
4847
shell: bash
4948

5049
- name: Publish a Python distribution to PyPI
50+
if: ${{ github.event_name == 'create' && github.event.ref_type == 'tag' }}
5151
uses: pypa/gh-action-pypi-publish@master
5252
with:
5353
user: __token__

.github/workflows/test_code.yml

Lines changed: 6 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,18 @@
11
name: Code Tests
22

33
on:
4-
push:
5-
branches:
6-
- main
74
pull_request:
8-
branches:
9-
- main
5+
push:
6+
branches: [main]
107

118
jobs:
129
run:
1310
runs-on: ${{ matrix.os }}
1411
strategy:
1512
matrix:
1613
os: [ubuntu-latest, windows-latest]
17-
python-version: [3.6, 3.9]
14+
python-version: ["3.6", "3.9"]
15+
fail-fast: false
1816

1917
steps:
2018
- uses: actions/checkout@v2
@@ -23,33 +21,18 @@ jobs:
2321
uses: s-weigand/setup-conda@v1
2422
with:
2523
activate-conda: false
24+
update-conda: true
2625
conda-channels: conda-forge
2726

2827
- name: Python ${{ matrix.python-version }}
2928
shell: bash -l {0}
3029
run: |
31-
conda create --yes --name TEST python=${{ matrix.python-version }} pip --file requirements.txt --file requirements-dev.txt --channel conda-forge --strict-channel-priority
30+
conda create --yes --name TEST python=${{ matrix.python-version }} pip --file requirements.txt --file requirements-dev.txt
3231
source activate TEST
3332
pip install -e . --no-deps --force-reinstall
3433
35-
- name: Tarball tests
36-
shell: bash -l {0}
37-
run: |
38-
source activate TEST
39-
pip wheel . -w dist --no-deps
40-
check-manifest --verbose
41-
twine check dist/*
42-
4334
- name: Tests
4435
shell: bash -l {0}
4536
run: |
4637
source activate TEST
4738
pytest -vv --ignore=tests/selenium
48-
49-
- name: Tests with latest branca
50-
shell: bash -l {0}
51-
run: |
52-
source activate TEST
53-
conda remove branca --yes
54-
pip install git+https://github.com/python-visualization/branca.git
55-
pytest -vv --ignore=tests/selenium
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
name: Code Tests with Latest branca
2+
3+
on:
4+
pull_request:
5+
push:
6+
branches: [main]
7+
8+
jobs:
9+
run:
10+
runs-on: ubuntu-latest
11+
12+
steps:
13+
- uses: actions/checkout@v2
14+
15+
- name: Setup Conda
16+
uses: s-weigand/setup-conda@v1
17+
with:
18+
activate-conda: false
19+
update-conda: true
20+
conda-channels: conda-forge
21+
22+
- name: Python 3.x
23+
shell: bash -l {0}
24+
run: |
25+
conda create --yes --name TEST python=3 pip --file requirements.txt --file requirements-dev.txt
26+
source activate TEST
27+
pip install -e . --no-deps --force-reinstall
28+
29+
- name: Tests with latest branca
30+
shell: bash -l {0}
31+
run: |
32+
source activate TEST
33+
conda remove branca --yes
34+
pip install git+https://github.com/python-visualization/branca.git
35+
pytest -vv --ignore=tests/selenium

.github/workflows/test_selenium.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,14 @@ jobs:
1515
steps:
1616
- uses: actions/checkout@v2
1717
- name: Conda setup
18-
uses: s-weigand/setup-conda@v1.0.5
18+
uses: s-weigand/setup-conda@v1
1919
with:
2020
update-conda: true
2121

2222
- name: Conda environment creation
2323
shell: bash -l {0}
2424
run: |
25-
conda create --yes --name TEST python=3.9 pip --file requirements.txt --file requirements-dev.txt --channel conda-forge --strict-channel-priority
25+
conda create --yes --name TEST python=3 pip --file requirements.txt --file requirements-dev.txt --channel conda-forge --strict-channel-priority
2626
source activate TEST
2727
pip install -e . --no-deps --force-reinstall
2828

0 commit comments

Comments
 (0)