Skip to content

QVAC-24197 chore: drop the archived qvac-test-suite repo from the DevOps roster - #4452

Open
Victor-Rodzko wants to merge 1 commit into
mainfrom
chore/QVAC-24197-retire-test-suite-repo-refs
Open

Victor-Rodzko wants to merge 1 commit into
mainfrom
chore/QVAC-24197-retire-test-suite-repo-refs

Conversation

@Victor-Rodzko

Copy link
Copy Markdown
Contributor

🎯 What problem does this PR solve?

  • The test-suite framework moved into this monorepo as packages/test-suite in QVAC-23575 chore[bc]: add @qvac/test-suite as a monorepo package #4082 and has published from here since 0.11.0. tetherto/qvac-test-suite is being archived.
  • The DevOps PR dashboard still watches that repository. It has no open pull requests or issues, and once the repository is read-only none can ever be opened — so the entry is a guaranteed-empty query on every dashboard run, and a reader of the roster is told to expect work there that cannot exist.

Companion to tetherto/qvac-test-suite#94 (pointer README) and the npm deprecation of @qvac/qvac-test-suite. Same Asana task, QVAC-24197.

📝 How does it solve it?

  • Removes tetherto/qvac-test-suite from extraRepos in .github/teams/devops.json — the roster the dashboard actually reads.
  • Removes it from the mirrored prose list in .agents/skills/qv-devops-pr-status/SKILL.md, so the documented roster and the JSON don't drift.

These are the two references the fold deliberately left in place until the repository was ready to be archived.

The remaining mentions of the old repository are intentional and stay:

  • # Originally from qvac-test-suite provenance comments in .github/workflows/test-node-sdk.yml and .github/workflows/test-android-sdk.yml
  • the note in .github/workflows/trigger-reusable-lib-test-suite.yml explaining why the package publishes prebuilt output
  • the migration record in packages/test-suite/CHANGELOG.md and packages/test-suite/README.md
  • the old package names @qvac/qvac-test-suite / @tetherto/qvac-test-suite, which build-consumer-mobile.ts and test-loader.ts still resolve on purpose so a partially migrated consumer works

🧪 How was it tested?

  • .github/teams/devops.json parses; extraRepos is 25 entries and no longer contains tetherto/qvac-test-suite
  • Cross-checked the JSON roster against the list in SKILL.md — the two agree entry for entry
  • git grep qvac-test-suite — every remaining hit is one of the intentional cases listed above

🤖 Generated with Claude Code

The framework moved into this monorepo as `packages/test-suite` in #4082 and has
published from here since 0.11.0. The source repository is being archived, so the
DevOps PR dashboard no longer has anything to watch there — it has no open pull
requests or issues, and none can be opened once the repository is read-only.

`extraRepos` in `.github/teams/devops.json` is the roster the dashboard actually
reads; the list in the `qv-devops-pr-status` skill mirrors it in prose. Both are
updated together so they do not drift.

The remaining mentions of the old repository are deliberate and stay: the
`# Originally from qvac-test-suite` provenance comments in the two SDK test
workflows, the note in `trigger-reusable-lib-test-suite.yml` explaining why the
package publishes prebuilt output, and the migration record in the package
CHANGELOG and README.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Review Status

Current Status: ❌ PENDING
Approvals so far: none

Pending reviews: Needs 1 Management or Team Lead, and 1 more from Management, Team Lead, or Member.

@github-actions

Copy link
Copy Markdown
Contributor

License compliance — clean

No new dependency license findings in this PR.

Warn-only (shadow) mode — this check does not block merges yet.

Updated automatically by the canonical license compliance workflow.

NOTICE presence (advisory)

Missing NOTICE (advisory, does not block):

  • ./docs/website
  • ./packages/fabric/test/integration
  • ./packages/llm-llamacpp/benchmarks/server
  • ./packages/llm-llamacpp/benchmarks/performance
  • ./packages/inference-addon-cpp/mobile
  • ./packages/asr-ggml/benchmarks/server
  • ./packages/embed-llamacpp/benchmarks/server
  • ./packages/embed-llamacpp/benchmarks/performance
  • ./packages/sdk/e2e
  • ./packages/vla-ggml/sim/server
  • ./.github/actions/release-merge-guard

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