Skip to content

ci(os-49): remove obsolete shadow PR workflows - #1161

Merged
jtoelke2 merged 1 commit into
mainfrom
jtoelke/os-130-phase6-shadow-cleanup
May 5, 2026
Merged

ci(os-49): remove obsolete shadow PR workflows#1161
jtoelke2 merged 1 commit into
mainfrom
jtoelke/os-130-phase6-shadow-cleanup

Conversation

@jtoelke2

@jtoelke2 jtoelke2 commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the Phase 5 shadow PR workflows that still triggered on pull-request/* after the Phase 6 cutover.
  • Keep reusable/manual shadow workflows that are still useful, including shadow-rust-native-build.yml, shadow-docker-build.yml, and shadow-shared-cpu-spike.yml.
  • Update CI architecture docs to describe the post-cutover state.
  • Refresh mise.lock with mise lock so the new lockfile check passes on the current tool metadata.

Related Issue

  • OS-49

Changes

  • Delete shadow-branch-checks.yml, shadow-branch-e2e.yml, and shadow-ci-image.yml.
  • Remove stale docs references to the deleted PR-triggered shadow workflows.
  • Update Python standalone lockfile entries from the current mise lock output.

Testing

  • git diff --check passed.
  • Confirmed no stale references remain with rg.
  • mise lock was run with network access and matches the CI lockfile diff.
  • mise run pre-commit was run; it failed in existing Rust test ssh::tests::launch_editor_returns_friendly_error_when_binary_missing, unrelated to these workflow/doc/lockfile changes. The single test reproduces locally with mise x -- cargo test -p openshell-cli ssh::tests::launch_editor_returns_friendly_error_when_binary_missing -- --nocapture.

Checklist

  • Commit messages follow Conventional Commits.
  • Scope limited to Phase 6 shadow workflow cleanup plus required lockfile refresh.
  • Documentation updated for the workflow removal.

@jtoelke2
jtoelke2 requested a review from a team as a code owner May 5, 2026 00:56
drew
drew previously approved these changes May 5, 2026
@jtoelke2 jtoelke2 mentioned this pull request May 5, 2026
7 tasks
@jtoelke2
jtoelke2 force-pushed the jtoelke/os-130-phase6-shadow-cleanup branch from 96d14e1 to 62fe0d1 Compare May 5, 2026 02:46
@jtoelke2
jtoelke2 merged commit 9c53d12 into main May 5, 2026
24 checks passed
@jtoelke2
jtoelke2 deleted the jtoelke/os-130-phase6-shadow-cleanup branch May 5, 2026 03:14
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.

2 participants