Skip to content

Commit 732eca8

Browse files
committed
fix: place nosemgrep marker directly above the checkout step
Semgrep's inline-ignore directive only applies when the comment immediately precedes the matching line, with no other comments in between. Move the nosemgrep tag to the line above the actions/checkout step and keep the rationale comments above it.
1 parent 9b48a4f commit 732eca8

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/publish-pypi.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,11 @@ jobs:
2323
runs-on: ubuntu-latest
2424

2525
steps:
26-
# nosemgrep: yaml.github-actions.security.workflow-run-target-code-checkout.workflow-run-target-code-checkout
2726
# The job's `if` already gates on workflow_run.head_branch == 'main'
2827
# and workflow_run.event != 'pull_request', so a fork PR head can
2928
# never reach this checkout. We pin to workflow_run.head_sha to
3029
# publish exactly the commit that passed CI on main.
30+
# nosemgrep: yaml.github-actions.security.workflow-run-target-code-checkout.workflow-run-target-code-checkout
3131
- name: Checkout the exact commit that passed CI
3232
uses: actions/checkout@v4
3333
with:

0 commit comments

Comments
 (0)