Skip to content

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

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)#26
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 because the newly added .githooks/ directory violates the root shape allowlist and the K9 contract validation failed due to missing magic numbers and pedigree blocks in example files.

Overview

Two failures occurred in CI: a repository root shape check failure due to an unallowed .githooks/ directory, and a contract validation failure where scanned example files are missing required K9 headers.

Failures

Root Shape Check Failure (confidence: high)

  • Type: build
  • Affected jobs: 90322148507
  • Related to change: yes
  • Root cause: The script scripts/check-root-shape.sh detected .githooks/ as an unauthorized entry in the repository root directory.
  • Suggested fix: Either move the .githooks directory into an appropriate subdirectory or add an entry for it in .machine_readable/root-allow.txt.

K9 Contract Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90322146575
  • Related to change: yes
  • Root cause: The script .githooks/validate-k9.sh failed during validation because files under ./.machine_readable/self-validating/examples/ lack the required 'K9!' magic number on the first non-empty line and do not contain a pedigree = { ... } section.
  • Suggested fix: Update the K9 contract files in ./.machine_readable/self-validating/examples/ to include the 'K9!' magic number as the first non-empty line and add the required pedigree = { ... } block.

Summary

  • Change-related failures: 2 failures (Root shape check failure from unauthorized .githooks/ directory, and K9 contract validation errors in example files)
  • Infrastructure/flaky failures: 0 infrastructure or flaky failures
  • Recommended action: Move or allowlist the .githooks/ directory and fix the K9 contract headers/pedigree blocks in the example files.
Code Review ✅ Approved

Removes the uninstantiated and permanently failing e2e.yml workflow file from CI. 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

@hyperpolymath
hyperpolymath merged commit 877b795 into main Jul 28, 2026
43 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant