Skip to content

ci: remove CodeQL and fix Checks workflow#6

Merged
notque merged 1 commit into
mainfrom
fix/ci-workflows
May 7, 2026
Merged

ci: remove CodeQL and fix Checks workflow#6
notque merged 1 commit into
mainfrom
fix/ci-workflows

Conversation

@notque
Copy link
Copy Markdown
Owner

@notque notque commented May 7, 2026

Summary

  • Remove CodeQL workflow — requires "code scanning" enabled in repo settings. Will work automatically when repo goes public (free for public repos). Can re-add later.
  • Fix Checks workflow — removed 3 steps that called non-existent Makefile targets (check-dependency-licenses, check-addlicense, run-shellcheck). These were generated by go-makefile-maker but the corresponding features aren't configured.
  • REUSE compliance — added SPDX headers to README.md and CONTRIBUTING.md, added build/** and .claude/** to REUSE.toml annotations.

Test plan

  • make build-all passes
  • make run-golangci-lint — 0 issues
  • reuse lint — compliant
  • typos . — clean
  • CI Checks workflow passes (verify after merge)

- Remove CodeQL workflow (requires code scanning enabled; works automatically
  when repo goes public)
- Remove check-dependency-licenses and check-addlicense steps (targets don't
  exist in Makefile)
- Remove shellcheck step (no shell scripts in repo)
- Add SPDX headers to README.md and CONTRIBUTING.md for REUSE compliance
- Add build/** and .claude/** to REUSE.toml annotations
@notque notque merged commit 3b70030 into main May 7, 2026
4 checks passed
@notque notque deleted the fix/ci-workflows branch May 7, 2026 12:20
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