Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
18 changes: 10 additions & 8 deletions .github/workflows/consumer-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ on:
required: false
default: main
pipeline-python-ref:
description: Branch, tag or SHA of pipeline-python to test against
description: Branch, tag or SHA of 51degrees-python to test against
required: false
default: main
pipeline-php-core-ref:
Expand Down Expand Up @@ -218,33 +218,35 @@ jobs:
npx jest tests/javascriptbuilder.test.js

pipeline-python:
name: pipeline-python JavaScript builder
name: 51degrees-python JavaScript builder
runs-on: ubuntu-latest
steps:
- name: Check out the template under review
uses: actions/checkout@v4
with:
path: template

- name: Check out pipeline-python
- name: Check out 51degrees-python
uses: actions/checkout@v4
with:
repository: 51Degrees/pipeline-python
repository: 51Degrees/51degrees-python
ref: ${{ inputs.pipeline-python-ref || 'main' }}
submodules: recursive
# The Pipeline API packages are under pipeline/ and need only the
# direct submodules, not the data files under cxx/.
submodules: true
path: consumer

- name: Swap in the template under review
run: |
set -euo pipefail
dest=consumer/fiftyone_pipeline_core/src/fiftyone_pipeline_core/js_templates
dest=consumer/pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/js_templates
if [ ! -d "$dest" ]; then
echo "::error::$dest is missing - the submodule has moved and this workflow needs updating"
exit 1
fi
cp template/*.mustache "$dest/"
if git -C "$dest" diff --quiet; then
echo "::notice::identical to the revision pipeline-python pins, nothing swapped in"
echo "::notice::identical to the revision 51degrees-python pins, nothing swapped in"
else
git -C "$dest" --no-pager diff --stat
fi
Expand All @@ -254,7 +256,7 @@ jobs:
python-version: '3.13'

- name: Run the JavaScript builder tests
working-directory: consumer/fiftyone_pipeline_core
working-directory: consumer/pipeline/fiftyone_pipeline_core
run: |
set -euo pipefail
python -m pip install --upgrade pip
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ This repo is not a stand-alone package, but is shipped as part of and used by ea
- [pipeline-dotnet](https://github.com/51Degrees/pipeline-dotnet) as a submodule
- [pipeline-java](https://github.com/51Degrees/pipeline-java) as a submodule
- [pipeline-node](https://github.com/51Degrees/pipeline-node) as a submodule
- [pipeline-python](https://github.com/51Degrees/pipeline-python) as a submodule
- [51degrees-python](https://github.com/51Degrees/51degrees-python) as a submodule, under `pipeline/`
- [pipeline-php-core](https://github.com/51Degrees/pipeline-php-core) as a static dependency

Wherever it is a submodule it will be updated by `Nightly Submodule Update` action, wherever it is a static dependency it will be updated by the `Nightly Package Update` action within a target repository.
Expand Down
Loading