Fix CI action pinning policy violation - #5
Merged
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuidePins all GitHub Actions in the CI workflow to specific commit SHAs to satisfy action pinning policies and removes unused secret environment variables from the test step. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Since the actions are now pinned to specific SHAs, consider adding a brief note or reference in the workflow (or a central place) about how and when to update these pins so they don’t silently drift out-of-date over time.
- You removed the OPENAI/GROQ/QDRANT env vars from the test step; if any tests still depend on external APIs, consider reintroducing them in a more tightly scoped way (e.g., only for specific jobs or steps) rather than globally or not at all.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Since the actions are now pinned to specific SHAs, consider adding a brief note or reference in the workflow (or a central place) about how and when to update these pins so they don’t silently drift out-of-date over time.
- You removed the OPENAI/GROQ/QDRANT env vars from the test step; if any tests still depend on external APIs, consider reintroducing them in a more tightly scoped way (e.g., only for specific jobs or steps) rather than globally or not at all.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
lioneldyla
merged commit Jun 30, 2026
342ce6a
into
codex/advanced-rag-contracts
1 of 3 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR pins the CI workflow actions to full commit SHAs so the validate job can pass repository policy checks.
Summary by Sourcery
Pin CI workflow actions to specific commit SHAs and simplify test step environment configuration.
CI: