Skip to content

fix(ci): make fake gates real and add Elixir lint gate - #699

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/ci-fake-gates-694
Aug 12, 2026
Merged

fix(ci): make fake gates real and add Elixir lint gate#699
hyperpolymath merged 1 commit into
mainfrom
fix/ci-fake-gates-694

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Fixes for issue #694:

  1. tests.yml: Fix E2E Rust CLI Scan - Changed 'hyper' to 'hypatia', removed '|| true'
  2. quality.yml: Fix lint job that cannot fail - Removed all '||' and continue-on-error
  3. tests.yml: Add Elixir lint gate - Added 'mix compile --warnings-as-errors' to e2e-elixir job

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe vibe@mistral.ai

Fixes for issue #694:

1. tests.yml: Fix E2E Rust CLI Scan
   - Changed 'hyper' to 'hypatia' (correct binary name)
   - Removed '|| true' so the step can fail properly

2. quality.yml: Fix lint job that cannot fail
   - Removed '|| true' from Check file permissions step
   - Removed '|| echo "None found"' from Check TODO/FIXME step
   - Removed '|| echo "No large files"' from Check for large files step
   - Removed 'continue-on-error: true' from EditorConfig check step
   - Removed '--include="*.res"' from grep (zero .res files in repo)

3. tests.yml: Add Elixir lint gate
   - Added 'mix compile --warnings-as-errors' step to e2e-elixir job
   - This addresses the 21 warnings mentioned in issue #694

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
@hyperpolymath
hyperpolymath merged commit 31e313d into main Aug 12, 2026
25 of 43 checks passed
@hyperpolymath
hyperpolymath deleted the fix/ci-fake-gates-694 branch August 12, 2026 00:14
@gitar-bot

gitar-bot Bot commented Aug 12, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

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