Skip to content

Conversation

@felickz
Copy link
Contributor

@felickz felickz commented Apr 30, 2025

This pull request introduces documentation updates for customizing the behavior of the UnpinnedActionsTag query in CodeQL. The changes include detailed instructions on how to extend the list of trusted Action publishers and a reference link to the customization guide. Inspired by lack of visibility here: #18316 (comment)

Documentation Enhancements:

Copilot AI review requested due to automatic review settings April 30, 2025 13:48
@felickz felickz requested a review from a team as a code owner April 30, 2025 13:48
@github-actions github-actions bot added documentation Actions Analysis of GitHub Actions labels Apr 30, 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.

Pull Request Overview

This pull request adds documentation to help users customize the behavior of the UnpinnedActionsTag query by providing detailed configuration instructions and examples.

  • Updated UnpinnedActionsTag.md to include a reference link to the customization guide.
  • Added a new file, UnpinnedActionsTag-CUSTOMIZING.md, with step-by-step configuration instructions and examples.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
actions/ql/src/Security/CWE-829/UnpinnedActionsTag.md Added reference to the customizing guide to inform users about extending the trusted Action publishers list.
actions/ql/src/Security/CWE-829/UnpinnedActionsTag-CUSTOMIZING.md New documentation file with configuration steps and examples for customizing trusted Action publishers.

@github-actions github-actions bot removed the Actions Analysis of GitHub Actions label May 1, 2025
@jonjanego
Copy link
Member

@felickz @adityasharad lets get this merged so it can be included with 2.21.3

@adityasharad adityasharad merged commit f814849 into github:main May 15, 2025
7 checks passed
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