Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| - name: Checkout repository | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ github.event.pull_request.head.sha }} | ||
| fetch-depth: 1 | ||
|
|
||
| - name: Fetch base commit for diff |
Check failure
Code scanning / CodeQL
Checkout of untrusted code in trusted context High
|
|
||
| - name: Run doc-pr review | ||
| if: steps.changed-files.outputs.count > 0 | ||
| uses: anthropics/claude-code-action@v1 |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
Move base_branch comparison into the same step that sets it, using the
shell variable directly instead of re-interpolating via ${{ }}. This
prevents potential code injection from untrusted PR branch names.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
| - name: Checkout repository | ||
| if: steps.pr-info.outputs.is_fork == 'false' && steps.pr-info.outputs.targets_dev == 'true' | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ steps.pr-info.outputs.branch }} | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install Vale |
Check failure
Code scanning / CodeQL
Checkout of untrusted code in trusted context High
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Issues in PR changes
.claude/skills/dale/SKILL.md
Line 24: Grammar error - "ones" should be "once". The sentence should read "Mark each Todo as complete once you've checked the given file for that rule."
To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).
To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).
Note: Automated fixes are only available for branches in this repository, not forks.
There was a problem hiding this comment.
Issues in PR changes
.claude/skills/dale/SKILL.md
Line 46: Missing trailing newline at end of file.
To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).
To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).
Note: Automated fixes are only available for branches in this repository, not forks.
|
|
||
| - name: Handle @claude request | ||
| if: steps.pr-info.outputs.is_fork == 'false' && steps.pr-info.outputs.targets_dev == 'true' | ||
| uses: anthropics/claude-code-action@v1 |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
There was a problem hiding this comment.
Issues in PR changes
Note: The user listed the following files for review, but they were NOT actually modified in this PR:
- docs/accessanalyzer/11.6/solutions/filesystem/overview.md
- docs/accessanalyzer/12.0/solutions/filesystem/overview.md
- docs/auditor/10.7/configuration/fileservers/windows/ports.md
- docs/auditor/10.7/configuration/logonactivity/ports.md
- docs/auditor/10.8/configuration/azurefiles/overview.md
- docs/auditor/10.8/configuration/fileservers/qumulo/overview.md
- docs/auditor/10.8/configuration/fileservers/windows/ports.md
- docs/auditor/10.8/configuration/logonactivity/ports.md
- docs/auditor/10.8/configuration/overview.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/FileSystem-Permissions-Format-Reference.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/SharePoint-Permissions-Format-Reference.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/access-analyzer-event-log-ids.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/access-analyzer-version-numbers.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/net_dependencies_for_netwrix_access_analyzer.md
- docs/kb/accessanalyzer/reference-and-technical-specifications/what-does-licensed-host-count-exceeded-mean-in-the-access-analyzer-banner.md
- docs/passwordsecure/9.3/index.md
The following files were actually modified in PR 420 and have been reviewed.
docs/CLAUDE.md
Line 125: Passive voice violation. "Fork PRs cannot be pushed to" should use active voice and be clearer. Change to "Automated fixes cannot be pushed to fork PRs."
All other files reviewed (.claude/agents/.md, .claude/skills//SKILL.md, CLAUDE.md) have no issues.
To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).
To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).
Note: Automated fixes are only available for branches in this repository, not forks.
1 participant
No description provided.