Skip to content

ci: remove commented-out test job - #2645

Merged
roottool merged 1 commit into
mainfrom
ci/remove-commented-test-job
Oct 14, 2026
Merged

roottool merged 1 commit into
mainfrom
ci/remove-commented-test-job

Conversation

@roottool

Copy link
Copy Markdown
Owner

Summary

Remove the commented-out test job and its commented reference in gatekeeper.

Motivation

The block had been disabled for a long time. Its actions/checkout was pinned to an old v5.0.0 SHA that is no longer tracked by Renovate, so it would be stale if enabled as is. When tests are introduced it needs to be rewritten anyway, and the previous version stays in git history.

Changes

  • Delete the commented test job, including the Codecov upload step.
  • Delete the commented # - test entry in the gatekeeper needs list.

Testing

  • Format check (bun run format, lint:dprint, lint:oxfmt)
  • Setup completes and Gatekeeper does not fail

Notes

Only ci.yml changes, so it matches no path filter and Build and Lint are expected to be skipped on this PR.

🤖 Generated with Claude Code

@roottool roottool self-assigned this Oct 14, 2026
@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 75241128-d6eb-4ef2-8389-21490c02a9bc
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying portfolio with  Cloudflare Pages  Cloudflare Pages

Latest commit: c74429e
Status: ✅  Deploy successful!
Preview URL: https://2ad3932d.portfolio-6od.pages.dev
Branch Preview URL: https://ci-remove-commented-test-job.portfolio-6od.pages.dev

View logs

@roottool
roottool merged commit 85896dc into main Oct 14, 2026
15 checks passed
@roottool
roottool deleted the ci/remove-commented-test-job branch October 14, 2026 22:46
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