Skip to content

Commit

Permalink
reorganize tests, make devdeps testing optional, and unpin latest Pyt…
Browse files Browse the repository at this point in the history
…hon testing (#213)

* separate devdeps into optional workflow

* update names

* fix branch name

* fix comment

* fix job filter

* fix repo name

* cache WebbPSF data for Romancal testing

* remove hyphens

* remove Windows builds

* remove WebbPSF data caching

* add more test runs to devdeps

---------

Co-authored-by: Howard Bushouse <[email protected]>
  • Loading branch information
zacharyburnett and hbushouse authored Dec 11, 2023
1 parent b179285 commit d71cb6d
Show file tree
Hide file tree
Showing 3 changed files with 52 additions and 12 deletions.
16 changes: 7 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: CI
name: test

on:
push:
Expand All @@ -8,11 +8,10 @@ on:
tags:
- "*"
pull_request:
branches:
- main
schedule:
# Weekly Monday 9AM build
- cron: "0 9 * * 1"
# * is a special character in YAML so you have to quote this string
- cron: '0 9 * * 1'

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
Expand All @@ -32,15 +31,14 @@ jobs:
- linux: py39-oldestdeps-cov-xdist
- linux: py39-xdist
- linux: py310-xdist
- linux: py311-xdist
- linux: py311-cov-xdist
coverage: codecov
- macos: py311-xdist
- linux: py3-cov-xdist
coverage: 'codecov'
test_downstream:
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@main
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@v1
with:
setenv: |
CRDS_PATH: /tmp/crds_cache
CRDS_PATH: /tmp/data/crds_cache
CRDS_CLIENT_RETRY_COUNT: 3
CRDS_CLIENT_RETRY_DELAY_SECONDS: 20
envs: |
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/ci_cron.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
name: Weekly cron
name: test on schedule

on:
schedule:
# Weekly Monday 6AM build
# Weekly Monday midnight build
- cron: "0 0 * * 1"
workflow_dispatch:

Expand All @@ -12,9 +12,9 @@ concurrency:

jobs:
test:
if: (github.repository == 'spacetelescope/stcal' && (github.event_name == 'schedule' || github.event_name == 'push' || github.event_name == 'workflow_dispatch' || contains(github.event.pull_request.labels.*.name, 'run scheduled tests')))
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@v1
with:
envs: |
- macos: py39-xdist
- macos: py310-xdist
- linux: py3-devdeps-xdist
42 changes: 42 additions & 0 deletions .github/workflows/tests_devdeps.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
name: test with development versions

on:
push:
branches:
- main
- '*x'
tags:
- '*'
pull_request:
schedule:
# Weekly Monday 9AM build
# * is a special character in YAML so you have to quote this string
- cron: '0 9 * * 1'
workflow_dispatch:

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
test:
if: (github.repository == 'spacetelescope/stcal' && (github.event_name == 'schedule' || github.event_name == 'push' || github.event_name == 'workflow_dispatch' || contains(github.event.pull_request.labels.*.name, 'run devdeps tests')))
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@v1
with:
envs: |
- linux: py39-devdeps-xdist
- linux: py310-devdeps-xdist
- linux: py311-devdeps-xdist
- linux: py3-devdeps-xdist
test_downstream:
if: (github.repository == 'spacetelescope/stcal' && (github.event_name == 'schedule' || github.event_name == 'push' || github.event_name == 'workflow_dispatch' || contains(github.event.pull_request.labels.*.name, 'run devdeps tests')))
uses: OpenAstronomy/github-actions-workflows/.github/workflows/tox.yml@v1
with:
setenv: |
CRDS_PATH: /tmp/data/crds_cache
CRDS_CLIENT_RETRY_COUNT: 3
CRDS_CLIENT_RETRY_DELAY_SECONDS: 20
envs: |
- linux: py3-jwst-devdeps-xdist
- linux: py3-romancal-devdeps-xdist

0 comments on commit d71cb6d

Please sign in to comment.