Skip to content

Conversation

@dixonjoel
Copy link
Collaborator

What does this Pull Request accomplish?

Eliminate one layer of workflow (run_unit_tests.yml) and put a call to run_and_upload_unit_tests composite action directly from test_package.yml.

Why should this Pull Request be merged?

We were getting four levels of workflow hierachy running unit tests. This is kind of unnecessary and hard to follow the flow of the CI.

What testing has been done?

Current PR checks and unit tests still run and pass.

Copilot AI review requested due to automatic review settings October 2, 2025 14:34
Copy link
Contributor

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 flattens the GitHub Actions workflow hierarchy by eliminating the run_unit_tests.yml workflow file and directly integrating its functionality into test_package.yml. This reduces the workflow call chain from four levels to three levels, making the CI flow easier to follow.

Key changes:

  • Removed intermediate run_unit_tests.yml workflow call
  • Added direct job steps for checkout, Python setup, and Poetry configuration
  • Integrated the run_and_upload_unit_tests composite action call directly

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2025

Test Results

  120 files  ±0    120 suites  ±0   3m 15s ⏱️ -10s
  240 tests ±0    238 ✅ ±0   2 💤 ±0  0 ❌ ±0 
2 400 runs  ±0  2 370 ✅ ±0  30 💤 ±0  0 ❌ ±0 

Results for commit 03c59bb. ± Comparison against base commit ca86811.

@dixonjoel dixonjoel merged commit 3e8b1fa into main Oct 2, 2025
347 checks passed
@dixonjoel dixonjoel deleted the users/jdixon/flatten-run-unit-tests branch October 2, 2025 19:51
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.

3 participants