Pewarta Proxy

📋 Server Request & Proxy Logs

Loading active proxy settings...
Developed by Khairul Adha (adhexa.id)
Mengambil log server...

⚙️ Pengaturan Proxy Target

Skip to content

Shared: Add missing QLDoc in variable capture library - #22955

Merged
owen-mc merged 1 commit into
github:mainfrom
owen-mc:shared/docs/variable-capture
Oct 30, 2026
Merged

owen-mc merged 1 commit into
github:mainfrom
owen-mc:shared/docs/variable-capture

Conversation

@owen-mc

@owen-mc owen-mc commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

This avoids CI failures for PRs implementing this library for new languages.

@owen-mc
owen-mc requested a review from a team as a code owner October 30, 2026 11:40
Copilot AI auto review requested due to automatic review settings October 30, 2026 11:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation-only change addresses missing QLDoc comments for variable-capture APIs and consistency checks, with no active review issues identified.

No open issues 🎉

What changed in this PR

This documentation-only pull request adds missing QLDoc comments to the shared variable-capture library, helping prevent documentation-related CI failures for new language integrations.

Changes:

  • Documents the BasicBlock and SynthesizedCaptureNode helper APIs.
  • Documents the ConsistencyChecks module and its validation predicates.
  • Clarifies the behavior and parameters of variable-capture consistency checks.
File Description
shared/​dataflow/​codeql/​dataflow/​VariableCapture.qll Adds QLDoc comments for variable-capture APIs and consistency checks.

Review effort: Lite (auto)

@owen-mc
owen-mc merged commit cd23825 into github:main Oct 30, 2026
107 of 108 checks passed
@owen-mc
owen-mc deleted the shared/docs/variable-capture branch October 30, 2026 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants