Skip to content

Commit

Permalink
Simplify CI (#34)
Browse files Browse the repository at this point in the history
* Simplify CI

* Fix test error caused by breaking change in EE API

It looks like the string repr for `ee.Projection` changed to
`ee.projection.Projection`, so my check for the fallback repr
started failing. Just updated that to match the new repr.
  • Loading branch information
aazuspan authored Mar 13, 2024
1 parent 145f0d6 commit c439adb
Show file tree
Hide file tree
Showing 4 changed files with 46 additions and 50 deletions.
9 changes: 0 additions & 9 deletions .github/scripts/make_ee_token.py

This file was deleted.

45 changes: 45 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
name: tests

on: push

jobs:
test:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [ '3.8', '3.9', '3.10', '3.11', '3.12' ]

steps:
- uses: actions/checkout@v3
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install hatch
- name: Write persistent EE credentials
env:
EE_TOKEN: ${{ secrets.EE_TOKEN }}
run: |
mkdir -p /home/runner/.config/earthengine
echo $EE_TOKEN > /home/runner/.config/earthengine/credentials
- name: Test with pytest
run: |
hatch run test:all
lint:
runs-on: ubuntu-latest

steps:
- name: Checkout code
uses: actions/checkout@v4

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install hatch
- name: Run pre-commit hooks
uses: pre-commit/[email protected]
40 changes: 0 additions & 40 deletions .github/workflows/tests.yml

This file was deleted.

2 changes: 1 addition & 1 deletion tests/test_reprs.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@ def test_error():
"""
with pytest.warns(UserWarning):
rep = ee.Projection("not a real epsg")._repr_html_()
assert "ee.Projection object" in rep
assert "Projection object" in rep

0 comments on commit c439adb

Please sign in to comment.