Skip to content

DEVOPS-977 test zizmor, null change#118

Open
andrewg-mira wants to merge 6 commits intodevelopfrom
DEVOPS-977-zizmor-test-3
Open

DEVOPS-977 test zizmor, null change#118
andrewg-mira wants to merge 6 commits intodevelopfrom
DEVOPS-977-zizmor-test-3

Conversation

@andrewg-mira
Copy link

No description provided.

Copilot AI review requested due to automatic review settings March 13, 2026 18:53
Copy link

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

Updates the repo to exercise/validate the zizmor-based security scanning workflow, with a small README modification.

Changes:

  • Add placeholder lines to the README.
  • Change the security scan workflow trigger to pull_request_target.
  • Pin reused CI workflows from CI-tools to a specific commit SHA.

Reviewed changes

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

File Description
README.rst Adds placeholder “Test” content in the “Why?” section.
.github/workflows/security_scan.yml Switches PR trigger type and pins reusable workflows to a commit SHA.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +58 to +59
Test
Test 2
@andrewg-mira andrewg-mira force-pushed the DEVOPS-977-zizmor-test-3 branch from a997ff1 to ec44723 Compare March 13, 2026 20:30
@andrewg-mira andrewg-mira force-pushed the DEVOPS-977-zizmor-test-3 branch from b0d5d7d to b830398 Compare March 16, 2026 14:04
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.

2 participants