Skip to content

Documentation Actions EnvironmentCheck - #22454

Open
knewbury01 wants to merge 5 commits into
github:mainfrom
knewbury01:knewbury01/customize-actions-sanitizers-documentation
Open

Documentation Actions EnvironmentCheck#22454
knewbury01 wants to merge 5 commits into
github:mainfrom
knewbury01:knewbury01/customize-actions-sanitizers-documentation

Conversation

@knewbury01

Copy link
Copy Markdown
Contributor

Improve documentation on actions/security/ControlChecks.qll EnvironmentCheck

currently was undocumented and as such its intended ways to use it could stand to be clarified

I dont think this needs a change note... but let me know if that is an incorrect assumption

@knewbury01
knewbury01 requested a review from a team as a code owner August 27, 2026 18:21
Copilot AI balanced review requested due to automatic review settings August 27, 2026 18:21

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.

Copilot review overview

Review tier: Balanced
Findings: 1 Low severity

New issues introduced by this change (1)
Severity Finding
Low severity actions/​ql/​lib/​codeql/​actions/​security/​ControlChecks.qll — Correct the spelling of “vulnerabilities” in the new class documentation. This issue also appears…
What changed in this PR

Documents how EnvironmentCheck treats deployment environments as configurable sanitizers.

Changes:

  • Explains default and externally configured behavior.
  • Documents disabling default sanitizer environments.
File Description
actions/​ql/​lib/​codeql/​actions/​security/​ControlChecks.qll Adds EnvironmentCheck documentation.
Suppressed comments (1)

actions/ql/lib/codeql/actions/security/ControlChecks.qll:287

  • Correct the spelling of “sanitizer” in the description of the empty-model behavior.
 * To create a default of "no default saniziter environment" one can add an

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread actions/ql/lib/codeql/actions/security/ControlChecks.qll Outdated
Comment thread actions/ql/lib/codeql/actions/security/ControlChecks.qll Fixed
Comment thread actions/ql/lib/codeql/actions/security/ControlChecks.qll Fixed
@github-actions github-actions Bot added the Actions Analysis of GitHub Actions label Aug 27, 2026
Comment thread actions/ql/lib/codeql/actions/security/ControlChecks.qll Outdated
@owen-mc owen-mc added the no-change-note-required This PR does not need a change note label Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Actions Analysis of GitHub Actions no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants