Skip to content

Commands to use Pipeline Class#456

Merged
pooleycodes merged 5 commits intomainfrom
implement-pipeline-phase-manage
Feb 16, 2026
Merged

Commands to use Pipeline Class#456
pooleycodes merged 5 commits intomainfrom
implement-pipeline-phase-manage

Conversation

@pooleycodes
Copy link
Copy Markdown
Contributor

@pooleycodes pooleycodes commented Jan 20, 2026

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

In transitioning to using the Pipeline class to manage the actual transform phase of the data, the final step in this process was to remove the phase imports used by commands.py for run pipeline().

There is also tests ported over that simulated the async service to use the new pipeline transform instead, once again simulating how the async service transforms as it has already moved from creating the phases in this way.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes - integration tests exist already, with acceptance for async testing and real data transformation comparison using S3 added as well
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

@pooleycodes pooleycodes changed the title commands.py run_pipline uses Pipeline Class, and real transfrom and u… Commands to use Pipeline Class Jan 20, 2026
@pooleycodes pooleycodes linked an issue Jan 20, 2026 that may be closed by this pull request
# Conflicts:
#	tests/acceptance/test_async_backend_pipeline_run.py
#	tests/integration/pipeline/test_main.py
@pooleycodes pooleycodes marked this pull request as ready for review February 6, 2026 16:16
@pooleycodes pooleycodes merged commit f187086 into main Feb 16, 2026
4 checks passed
@pooleycodes pooleycodes deleted the implement-pipeline-phase-manage branch February 16, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Pipeline Class in commands.py

2 participants