Skip to content

ci: add persist-credentials: false to checkout actions that don't push#813

Open
aristarkhovNV wants to merge 1 commit into
mainfrom
aaristarkhov/no-creds-checkout
Open

ci: add persist-credentials: false to checkout actions that don't push#813
aristarkhovNV wants to merge 1 commit into
mainfrom
aaristarkhov/no-creds-checkout

Conversation

@aristarkhovNV

@aristarkhovNV aristarkhovNV commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Description

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Testing

Checklist

  • I have read and understood the contribution guidelines
  • I have run the linter and formatter with SKIP=check-copyright-year pre-commit run --all-files
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix/feature works (or explained why not)
  • I have signed off all my commits (git commit -s) per the DCO

Summary by CodeRabbit

  • Security
    • Improved workflow security by preventing temporary checkout credentials from being retained after repository checkout.
    • Applied this protection to documentation previews, documentation builds, and pre-commit checks.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main CI change: disabling persisted checkout credentials in workflows that do not push.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch aaristarkhov/no-creds-checkout

Comment @coderabbitai help to get the list of available commands.

@aristarkhovNV
aristarkhovNV marked this pull request as ready for review July 22, 2026 21:42
@aristarkhovNV
aristarkhovNV enabled auto-merge (squash) July 22, 2026 21:43
@aristarkhovNV
aristarkhovNV requested a review from jiwenc-nv July 22, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant