Skip to content

ci: Update docker build workflow for uv workspaces#180

Merged
toby-coleman merged 1 commit intomainfrom
ci/update-docker-ci-for-uv-workspace
Dec 9, 2025
Merged

ci: Update docker build workflow for uv workspaces#180
toby-coleman merged 1 commit intomainfrom
ci/update-docker-ci-for-uv-workspace

Conversation

@toby-coleman
Copy link
Contributor

Summary

The Docker build CI stage needs modification to allow it to provide the correct requirements.txt before building the Docker image.

Changes

  • Update the uv export command as required.

@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Benchmark comparison for fd99eb3f (base) vs 08c4350c (PR)


------------------------------------------------------------------------------------------------------------------ benchmark: 2 tests -----------------------------------------------------------------------------------------------------------------
Name (time in ms)                                                                         Min                 Max                Mean            StdDev              Median               IQR            Outliers     OPS            Rounds  Iterations
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
test_benchmark_process_run (main/.benchmarks/Linux-CPython-3.12-64bit/0001_base)     442.5509 (1.0)      451.8016 (1.0)      448.3799 (1.0)      3.6748 (1.0)      448.8545 (1.0)      4.9620 (1.0)           1;0  2.2303 (1.0)           5           1
test_benchmark_process_run (pr/.benchmarks/Linux-CPython-3.12-64bit/0001_pr)         449.2087 (1.02)     461.2365 (1.02)     454.4158 (1.01)     4.4641 (1.21)     454.5239 (1.01)     5.2777 (1.06)          2;0  2.2006 (0.99)          5           1
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Legend:
  Outliers: 1 Standard Deviation from Mean; 1.5 IQR (InterQuartile Range) from 1st Quartile and 3rd Quartile.
  OPS: Operations Per Second, computed as 1 / Mean

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Docker build workflow to properly support uv workspaces by modifying the uv export command flags used when generating the requirements.txt file for Docker builds.

  • Updates uv export command flags to support workspace configuration
  • Replaces deprecated --no-dev with --no-default-groups to exclude development dependencies
  • Adds --all-packages to export all workspace members (plugboard and plugboard-schemas)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@toby-coleman toby-coleman merged commit fd01746 into main Dec 9, 2025
34 of 37 checks passed
@toby-coleman toby-coleman deleted the ci/update-docker-ci-for-uv-workspace branch December 9, 2025 20: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.

1 participant