Skip to content

ci: Remove empty codeql workflow file#1054

Open
s373nZ wants to merge 1 commit intomasterfrom
remove-codeql-workflow
Open

ci: Remove empty codeql workflow file#1054
s373nZ wants to merge 1 commit intomasterfrom
remove-codeql-workflow

Conversation

@s373nZ
Copy link
Copy Markdown

@s373nZ s373nZ commented Apr 27, 2026

Should remove the consistent GitHub Actions failures for inability to parse an empty file.

Summary by CodeRabbit

  • Chores
    • Internal workflow file formatting cleanup.

@s373nZ s373nZ self-assigned this Apr 27, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9fb5ad4f-092f-4404-8ca1-fa98840bc28a

📥 Commits

Reviewing files that changed from the base of the PR and between 8e00526 and 9cceab5.

📒 Files selected for processing (1)
  • .github/workflows/codeql.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/codeql.yml

Walkthrough

Removes a single blank line from the CodeQL workflow configuration file. No functional changes to the workflow logic or behavior are introduced.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/codeql.yml
Blank line removed; no functional changes to workflow logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing an empty CodeQL workflow file to resolve GitHub Actions parsing failures.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-codeql-workflow

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 and usage tips.

Copy link
Copy Markdown
Contributor

@arkanaai arkanaai Bot left a comment

Choose a reason for hiding this comment

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

LGTM — Approved

Trivial CI cleanup. The file .github/workflows/codeql.yml has been empty (single newline) since PR #304 intentionally disabled the CodeQL workflow. GitHub Actions can't parse an empty YAML file, producing recurring CI failures. Deleting it is the correct fix.

Checklist:

  • No protocol-critical code touched
  • No API/type/proto changes — zero cross-repo impact
  • No security implications
  • Git history confirms the file was intentionally emptied; full deletion is the natural next step

Ship it.


Reviewed by Arkana 🔍

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