Skip to content

fix(ci): explicitly pass uv.lock to OSV-Scanner - #41

Merged
alain-sv merged 1 commit into
developfrom
fix/osv-scanner-lockfile
May 13, 2026
Merged

fix(ci): explicitly pass uv.lock to OSV-Scanner#41
alain-sv merged 1 commit into
developfrom
fix/osv-scanner-lockfile

Conversation

@alain-sv

Copy link
Copy Markdown
Contributor

No description provided.

@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Add explicit UV lock file argument to OSV-Scanner workflows

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Explicitly pass uv.lock to OSV-Scanner in security workflows
• Add --lockfile=uv.lock argument to both scheduled and PR scan jobs
• Ensures OSV-Scanner correctly identifies and scans UV lock file dependencies
Diagram
flowchart LR
  A["OSV-Scanner Action"] -->|"scan-args"| B["--lockfile=uv.lock"]
  B -->|"scans"| C["uv.lock"]
  C -->|"identifies"| D["Dependencies"]
Loading

Grey Divider

File Changes

1. .github/workflows/security-scan.yml ⚙️ Configuration changes +2/-0

Add explicit UV lock file argument to OSV-Scanner

• Added --lockfile=uv.lock argument to the scan-scheduled job's scan-args
• Added --lockfile=uv.lock argument to the scan-pr job's scan-args
• Ensures OSV-Scanner explicitly processes the UV lock file for dependency scanning

.github/workflows/security-scan.yml


Grey Divider

ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented May 13, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan

Qodo Logo

@alain-sv
alain-sv merged commit 8481732 into develop May 13, 2026
9 checks passed
@alain-sv
alain-sv deleted the fix/osv-scanner-lockfile branch May 13, 2026 13:18
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