Skip to content

Conversation

@JarLob
Copy link
Contributor

@JarLob JarLob commented Jan 22, 2025

Pull Request checklist

All query authors

Internal query authors only

  • Autofixes generated based on these changes are valid, only needed if this PR makes significant changes to .ql, .qll, or .qhelp files. See the documentation (internal access required).
  • Changes are validated at scale (internal access required).
  • Adding a new query? Consider also adding the query to autofix.

Copilot AI review requested due to automatic review settings January 22, 2025 13:56
@JarLob JarLob requested a review from a team as a code owner January 22, 2025 13:56
@github-actions github-actions bot added documentation Actions Analysis of GitHub Actions labels Jan 22, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • actions/ql/test/query-tests/Security/CWE-1395/UseOfKnownVulnerableAction.expected: Language not supported

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@JarLob JarLob changed the title Fixed version range for known vulnerable actions Actions: Fix version range for known vulnerable actions Jan 22, 2025
Copy link
Member

@RasmusWL RasmusWL left a comment

Choose a reason for hiding this comment

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

Besides wording of the change-note, this LGTM 👍

Co-authored-by: Rasmus Wriedt Larsen <rasmuswriedtlarsen@gmail.com>
@RasmusWL RasmusWL merged commit bafed4e into github:main Jan 24, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Actions Analysis of GitHub Actions documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants