test: remove smoke-test fixture (12.7)#4
Conversation
|
Warning Rate limit exceeded
To unlock additional reviews beyond the hourly cap, upgrade to Pro or higher. Account admins can upgrade under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
WalkthroughA YAML smoke-test fixture file for phpvms was deleted, removing its package metadata, runtime requirements (PHP ^8.2, phpvms ^7.0), version 0.1.0 release information including zip artifact coordinates and SHA256, and its revocation status. ChangesFixture Removal
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour 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 |
Registry validation: failed[FAIL]
|
Without --diff-filter the validator picks up paths the PR removed and crashes with ENOENT in runPackageChecks when it tries to read them. Use --diff-filter=ACMRT so deleted entries never reach the checker; renamed/copied entries surface their new path which is what we want to validate. Refactor changedYamlFiles into a pure filterPackageYamlPaths helper so we can unit-test the path filter without a git repo. Also gate main() on direct execution so the test file can import the helper without the script auto-running. Caught while exercising task 12.7 of bootstrap-addon-registry: the smoke-test fixture removal PR (#4) failed validation with the ENOENT crash described above.
Final smoke-test step. After merge, publish.yml should rebuild the index without phpvms/smoke-test, leaving raw/packages.json empty (no other packages are registered yet).
129a753 to
53704c2
Compare
Final cleanup of the bootstrap smoke test. Removes
packages/phpvms/smoke-test.ymland verifies the index rebuilds without it.