Skip to content

test: revoke smoke-test fixture (12.6) - #3

Merged
nabeelio merged 1 commit into
mainfrom
smoke-test/revoke
May 5, 2026
Merged

test: revoke smoke-test fixture (12.6)#3
nabeelio merged 1 commit into
mainfrom
smoke-test/revoke

Conversation

@nabeelio

@nabeelio nabeelio commented May 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Marks phpvms/smoke-test as revoked: true with reason.
  • Exercises validator skip-upstream-checks-on-revoked rule (5.18).
  • After merge, discovery should skip this package on subsequent runs.

Test plan

  1. validate-pr should PASS without re-fetching the upstream zip.
  2. After merge, fire repository_dispatch; discovery should skip phpvms/smoke-test.

Verifies validator skips upstream/zip checks for revoked entries and
that discovery skips the package on subsequent runs.
@coderabbitai

coderabbitai Bot commented May 5, 2026

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: c6722a12-6502-43b6-bd15-cd1829510bb8

📥 Commits

Reviewing files that changed from the base of the PR and between 183068f and 9e38514.

📒 Files selected for processing (1)
  • packages/phpvms/smoke-test.yml

Walkthrough

The smoke-test fixture metadata is updated to mark the release as revoked, adding the revoked: true flag and a revoked_reason field to verify pipeline behavior with revoked releases.

Changes

Smoke-Test Fixture Update

Layer / File(s) Summary
Fixture Configuration
packages/phpvms/smoke-test.yml
Release metadata is marked as revoked with revoked: true and revoked_reason: Smoke-test fixture; revoked to verify pipeline behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A fixture small, so neat and brief,
Marks revocation beyond belief,
The smoke test hops through pipelines now,
With revoked flags—we'll test them, wow!
Pipeline truths in metadata show. ✨


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@phpvms-registry-bot

Copy link
Copy Markdown
Contributor

Registry validation: passed

[PASS] packages/phpvms/smoke-test.yml (upstream checks skipped — revoked)

@nabeelio
nabeelio merged commit 186fb37 into main May 5, 2026
3 checks passed
@nabeelio
nabeelio deleted the smoke-test/revoke branch May 5, 2026 01:34
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