Skip to content

ci/docs: correct CodeQL code-scanning caveat — repo is public - #412

Merged
M0LTE merged 1 commit into
mainfrom
claude/packet-net-security-testing-quwgjf
Jun 13, 2026
Merged

ci/docs: correct CodeQL code-scanning caveat — repo is public#412
M0LTE merged 1 commit into
mainfrom
claude/packet-net-security-testing-quwgjf

Conversation

@M0LTE

@M0LTE M0LTE commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

The repo is public (confirmed via the API — visibility: public), so CodeQL code scanning is free and available with no setup: the analyze step's SARIF upload just works, and results land in Security → Code scanning automatically. There is no GHAS/enable-it step to do — that caveat only applies to private repos.

This strips the stale "needs GitHub Advanced Security / enable code scanning" note from:

  • the codeql.yml header comment, and
  • the two CodeQL entries in docs/plan.md §17.

Deliberately not touched: the separate packet.net is private references in the plan (the self-update feed needing a public host, and the ax25-ts parity-check read token). Those are substantive design decisions authored elsewhere, not caveats — flagged for the owner to revisit rather than silently rewritten.

Comment/docs only; YAML validated.

https://claude.ai/code/session_01GSRLZXowGYeRieqaevEtBb


Generated by Claude Code

The repo is public (confirmed: visibility=public), so CodeQL code scanning is
free and available with no setup — the analyze step's SARIF upload just works.
Strip the stale "needs GitHub Advanced Security / enable code scanning" caveat
(that's the private-repo case) from the codeql.yml comment and the two CodeQL
plan entries.

Leaves untouched the separate, substantive "packet.net is private" design
references (self-update feed needing a public host; the ax25-ts parity-check
read token) — those are owner design decisions to revisit, not caveats to strip.

https://claude.ai/code/session_01GSRLZXowGYeRieqaevEtBb
@M0LTE
M0LTE merged commit 43c26b0 into main Jun 13, 2026
16 of 18 checks passed
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.

2 participants