Skip to content

chore(ci): remove un-instantiated e2e.yml (0 jobs, permanently failing) - #18

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/remove-uninstantiated-e2e
Jul 28, 2026
Merged

chore(ci): remove un-instantiated e2e.yml (0 jobs, permanently failing)#18
hyperpolymath merged 1 commit into
mainfrom
chore/remove-uninstantiated-e2e

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

This repos .github/workflows/e2e.ymlis an **un-instantiated scaffold**:jobs:is present but every job block is commented out. An emptyjobs:map is invalid, so the run dies with **0 jobs** andconclusion=failure— permanently, on every push touchingsrc/, ffi/ortests/**`.

It has never run a single test, so nothing is lost by removing it.

Why delete rather than instantiate: comparing git blob SHAs against a known-pure template clone shows this repos src/` is the RSR scaffold, differing in at most 4 files (largely just the repo name). There is no project-specific behaviour here to exercise end-to-end yet, and a job written anyway would pass without testing anything — a fake gate.

Debt recorded, not hidden: this repo has no E2E coverage. When real code lands, add a genuine E2E job and prove it can fail. The scaffold remains available in the template repo.

🤖 Generated with Claude Code

jobs: is present but every job block is commented out. An empty jobs: map is
invalid, so the run dies with 0 jobs and conclusion=failure on every push that
touches src/**, ffi/** or tests/**. It has never run a test.

Deleted rather than instantiated: blob-SHA comparison against a pure template
clone shows this repo's src/ differs from the scaffold in <=4 files, so there is
no project behaviour to exercise end-to-end. Writing a job anyway would pass
without testing anything -- a fake gate.

Debt: this repo has NO e2e coverage, and this file never provided any.

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

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown
CI failed: CI failed due to K9 contract validation errors and unauthorized root entries not present on the allowlist.

Overview

Two distinct validation failures occurred across CI jobs: K9 contract file validation errors and repository root shape check failures.

Failures

K9 Contract Validation Failed (confidence: high)

  • Type: tooling
  • Affected jobs: 90322229439
  • Related to change: unclear
  • Root cause: The K9 contract files fail validation checks due to missing the 'K9!' magic number, missing 'signature' or 'signature_required' fields, and missing or incomplete 'pedigree' blocks.
  • Suggested fix: Update all K9 contract files to include the exact 'K9!' magic number on their first non-empty line, include required 'signature' or 'signature_required' fields, and ensure valid pedigree blocks containing required fields.

Root Entry Allowlist Check Failed (confidence: high)

  • Type: configuration
  • Affected jobs: 90322231574
  • Related to change: yes
  • Root cause: The script scripts/check-root-shape.sh detected unauthorized files/directories in the repository root (.githooks/, .hypatia-ignore, CLAUDE.md, and flake.nix) which are not included in .machine_readable/root-allow.txt.
  • Suggested fix: Add .githooks/, .hypatia-ignore, CLAUDE.md, and flake.nix to .machine_readable/root-allow.txt or move them into appropriate subdirectories.

Summary

  • Change-related failures: 1 failure regarding root entry allowlist validation
  • Infrastructure/flaky failures: 0
  • Recommended action: Update .machine_readable/root-allow.txt to include the root entries, and fix the required K9 contract headers and pedigree blocks.
Code Review ✅ Approved

Removes the uninstantiated and permanently failing e2e.yml workflow file to clean up CI debt. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Jul 28, 2026
@hyperpolymath
hyperpolymath merged commit c75a035 into main Jul 28, 2026
41 of 45 checks passed
@hyperpolymath
hyperpolymath deleted the chore/remove-uninstantiated-e2e branch July 28, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant