Skip to content

Fix flaky benchmark collection in CI#73

Merged
admin-raintree merged 1 commit into
mainfrom
codex/fix-flaky-performance-ci
Jun 15, 2026
Merged

Fix flaky benchmark collection in CI#73
admin-raintree merged 1 commit into
mainfrom
codex/fix-flaky-performance-ci

Conversation

@admin-raintree

Copy link
Copy Markdown
Contributor

Summary

  • keep benchmark tests out of default CI coverage collection unless explicitly requested
  • preserve the existing DOCPULL_BENCHMARK_10K opt-in for the long 10k benchmark
  • allow the smaller benchmark suite to run when targeted directly or with DOCPULL_BENCHMARKS=1

Verification

  • uv run pytest --collect-only -q | rg "tests/benchmarks/test_performance.py|tests/benchmarks/test_10k_pages.py" || true
  • uv run pytest --cov=docpull --cov-report=term tests/benchmarks/test_performance.py::TestConversionPerformance::test_large_html_conversion -q -s
  • uv run pytest --cov=docpull --cov-report=xml --cov-report=term -q
  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy src
  • uv run --extra dev python -m pre_commit run --all-files --show-diff-on-failure

@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docpull Ready Ready Preview, Comment Jun 15, 2026 5:21am

Request Review

@admin-raintree admin-raintree merged commit e813231 into main Jun 15, 2026
17 checks passed
@admin-raintree admin-raintree deleted the codex/fix-flaky-performance-ci branch June 15, 2026 05:23
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