Skip to content

Conversation

@rtuck99
Copy link
Contributor

@rtuck99 rtuck99 commented Dec 18, 2025

Preparatory work for

Requires:

This updates the PlanRunner to make the move_to_udc_default_state and clean_up_udc plans something that is invoked using the same mechanism as the main load_centre_collect_full plan.

PlanRunner is further subclasses into an InProcessRunner in anticipation of implementing a sibling class which implements plan execution using a BlueAPI client to a remote server.

Instructions to reviewer on how to test:

  1. Do thing x
  2. Confirm thing y happens

Checks for reviewer

  • Would the PR title make sense to a user on a set of release notes

@rtuck99 rtuck99 changed the base branch from main to hyperion_on_blueapi December 18, 2025 15:42
@rtuck99 rtuck99 added the dev experience Changes relating to developer experience label Dec 18, 2025
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

❌ Patch coverage is 98.54015% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.30%. Comparing base (211fb63) to head (0769d9d).

Additional details and impacted files
@@                   Coverage Diff                   @@
##           hyperion_on_blueapi    #1531      +/-   ##
=======================================================
+ Coverage                92.29%   92.30%   +0.01%     
=======================================================
  Files                      143      144       +1     
  Lines                     8094     8135      +41     
=======================================================
+ Hits                      7470     7509      +39     
- Misses                     624      626       +2     
Components Coverage Δ
i24 SSX 78.44% <ø> (ø)
hyperion 97.89% <98.51%> (-0.05%) ⬇️
other 98.08% <100.00%> (+<0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev experience Changes relating to developer experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants