Skip to content

Commit

Permalink
Merge remote-tracking branch 'upstream/8.0.x' into 8.0.x-deconflict
Browse files Browse the repository at this point in the history
* upstream/8.0.x:
  Fix consolidation tutorial (cylc#5199)
  GH Actions Codecov upload: try using token arg (cylc#5179)
  • Loading branch information
wxtim committed Oct 25, 2022
2 parents 855b2bc + 1939932 commit 41ab911
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 4 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/test_fast.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,7 @@ jobs:
- os: 'macos-latest'
python-version: '3.7'
env:
PYTEST_ADDOPTS: --cov --cov-append -n 5 --color=yes -m "linktest or not linktest"

PYTEST_ADDOPTS: --cov --cov-append -n 5 --color=yes
steps:
- name: Checkout
uses: actions/checkout@v3
Expand Down Expand Up @@ -112,3 +111,9 @@ jobs:
name: '${{ github.workflow }} ${{ matrix.os }} py-${{ matrix.python-version }}'
flags: fast-tests
fail_ci_if_error: true
verbose: true
token: ${{ secrets.CODECOV_TOKEN }} # Token not required for public repos, but might reduce chance of random 404 error?

- name: Linkcheck
if: startsWith(matrix.os, 'ubuntu')
run: pytest -m linkcheck tests/unit
4 changes: 3 additions & 1 deletion .github/workflows/test_functional.yml
Original file line number Diff line number Diff line change
Expand Up @@ -299,4 +299,6 @@ jobs:
with:
name: '${{ github.workflow }} ${{ matrix.name }} ${{ matrix.chunk }}'
flags: functional-tests
fail_ci_if_error: false
fail_ci_if_error: true
verbose: true
token: ${{ secrets.CODECOV_TOKEN }} # Token not required for public repos, but might reduce chance of random 404 error?
3 changes: 3 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,9 @@ Maintenance release.
[#5196](https://github.com/cylc/cylc-flow/pull/5196) - Replace traceback
with warning, for scan errors where workflow is stopped.

[#5199](https://github.com/cylc/cylc-flow/pull/5199) - Fix a problem with
the consolidation tutorial.

-------------------------------------------------------------------------------
## __cylc-8.0.3 (<span actions:bind='release-date'>Released 2022-10-17</span>)__

Expand Down
2 changes: 1 addition & 1 deletion cylc/flow/etc/tutorial/consolidation-tutorial/flow.cylc
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
[[get_observations_aldergrove]]
script = get-observations
[[[environment]]]
SITE_ID = 3976
SITE_ID = 3917
API_KEY = DATAPOINT_API_KEY
[[get_observations_camborne]]
script = get-observations
Expand Down

0 comments on commit 41ab911

Please sign in to comment.