-
Notifications
You must be signed in to change notification settings - Fork 357
fix: update packages with detected vulnerabilities #1834
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file skipped due to size limits:
- poetry.lock
Looks good to me 🤙
💡 To request another review, post a new comment with "/windsurf-review".
|
| Branch | jselig/fix-dep-vulns |
| Testbed | ci-runner-linux |
⚠️ WARNING: No Threshold found!Without a Threshold, no Alerts will ever be generated.
Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the--ci-only-thresholdsflag.
Click to view all benchmark results
| Benchmark | Latency | seconds (s) |
|---|---|---|
| test/benchmarks/test_program.py::test_copy_everything_except_instructions | 📈 view plot | 10.49 s |
| test/benchmarks/test_program.py::test_instructions | 📈 view plot | 4.08 s |
| test/benchmarks/test_program.py::test_iteration | 📈 view plot | 4.12 s |
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
asaites
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not something that needs to be fixed here, but I think the $pyquil_version is meant to be passed as a build-arg from the Makefile for the docker target.
From https://github.com/rigetti/pyquil/actions/runs/17415859364/job/49444091708?pr=1833