Skip to content

fix: add uv dependency audit#316

Merged
qartik merged 4 commits into
mainfrom
codex-issue-312-audit-modernize
May 8, 2026
Merged

fix: add uv dependency audit#316
qartik merged 4 commits into
mainfrom
codex-issue-312-audit-modernize

Conversation

@qartik

@qartik qartik commented May 7, 2026

Copy link
Copy Markdown
Member

Summary

  • add uv audit --locked via make audit and run it before uv sync in CI
  • configure uv with a 7-day exclude-newer cooldown and document local audit usage
  • modernize project checks from pre-commit/mypy to prek/ty
  • remove the scheduled pre-commit auto-update workflow

Fixes #312

Validation

  • make audit
  • make lint
  • make tests

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds dependency vulnerability scanning via uv audit --locked to the project and CI, while also updating the developer tooling stack (switching from pre-commit/mypy to prek/ty) and removing now-obsolete mypy configuration.

Changes:

  • Add make audit (backed by uv audit --locked) and run it in CI before dependency sync.
  • Configure uv with a 7-day exclude-newer window and document local audit usage.
  • Replace mypy-based type-checking configuration with ty, and switch CI/lint commands from pre-commit to prek.

Reviewed changes

Copilot reviewed 18 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/test_wasm.py Remove per-file mypy error-code disable directive.
tests/test_phirgen.py Remove per-file mypy error-code disable directive.
tests/test_parallelization.py Remove per-file mypy error-code disable directive.
tests/test_parallel_tk2.py Remove per-file mypy error-code disable directive.
tests/test_api.py Remove per-file mypy error-code disable directive.
README.md Update dev setup to use prek; document dependency audit usage.
pytket/phir/sharding/sharder.py Simplify Bit extraction from command args; remove type-ignore.
pytket/phir/phirgen.py Remove per-file mypy error-code disable directive.
pytket/phir/phirgen_parallel.py Add AngleKey type alias to better type dict keys for angle grouping.
pytket/phir/cli.py Remove per-file mypy disable directive.
pytket/phir/api.py Remove per-file mypy error-code disable directive.
pyproject.toml Add prek/ty dev deps; configure uv exclude-newer; add ty config.
mypy.ini Remove mypy configuration file.
MANIFEST.in Stop including/excluding removed mypy.ini.
Makefile Switch install/dev flows to uv sync; add audit target; use prek for lint.
.pre-commit-config.yaml Bump hook revs; replace mypy hook with local ty hook.
.github/workflows/python-app.yml Run dependency audit step before uv sync; use prek in CI.
.github/workflows/pre-commit-au.yml Remove scheduled pre-commit auto-update workflow.

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

Comment thread .github/workflows/python-app.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@qartik qartik requested a review from Asa-Kosto-QTM May 7, 2026 16:25
@qartik qartik merged commit 698d7b0 into main May 8, 2026
10 checks passed
@qartik qartik deleted the codex-issue-312-audit-modernize branch May 8, 2026 12:30
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.

Add dependency vulnerability scanning with pip-audit or uv audit

2 participants