Skip to content

ci: contain schema-only test fanout#1701

Closed
lmeyerov wants to merge 1 commit into
masterfrom
ci/schema-only-test-silo
Closed

ci: contain schema-only test fanout#1701
lmeyerov wants to merge 1 commit into
masterfrom
ci/schema-only-test-silo

Conversation

@lmeyerov

@lmeyerov lmeyerov commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a conservative schema_only path classifier for public schema runtime, schema artifact, and public schema test paths.
  • Uses schema_only through narrow_python_only to skip unrelated docs, NetworkX, middle-version REST, graphviz, Neo4j, build, DGL, Spark, and AI/UMAP fanout for schema-only PRs.
  • Keeps schema-artifacts, test-gfql-core, test-pandas-compat-gfql, test-core-python, test-polars, and changed-line-coverage for schema runtime/artifact edits.
  • Keeps docs/source schema edits out of schema_only so docs-only schema changes run docs validation, and mixed schema code+docs fail broad.

Safety checks

  • git diff --check
  • Extracted changes-job shell block with bash -n
  • PyYAML parse of .github/workflows/ci.yml: 32 jobs
  • Credential/token scans clean
  • Source-derived scenario evaluator covering schema.py, schema artifacts, public schema test, schema docs, mixed schema code+docs, PlotterBase/shared surfaces, GFQL schema_effects, and workflow infra

Coverage notes

Schema-only still runs changed-line-coverage and explicitly keeps test-core-python/test-polars so coverage artifacts remain available. test-pandas-compat-gfql now includes graphistry/tests/compute/gfql/test_public_schema.py and runs for schema_lane to preserve pandas-version coverage for public schema behavior.

@lmeyerov lmeyerov marked this pull request as ready for review July 6, 2026 06:13
@lmeyerov lmeyerov marked this pull request as draft July 6, 2026 17:11
@lmeyerov

lmeyerov commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Closing after broader PR-history audit: this schema-only slice is safe but not historically ROI-supported right now. Last 500 merged PRs had no non-changelog schema_only hits, so we should hold off until a real schema PR/run shows over-fanout.

@lmeyerov lmeyerov closed this Jul 6, 2026
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