ci(profiles): W4 add coverage for 9 orphan extensions + mlops - #187
Conversation
- Add fastapi-observability (github-setup + fastapi-cors + fastapi-sqlalchemy + fastapi-opentelemetry + fastapi-rag-pgvector) covering 4 orphans with distinct categories (ci/security/database/observability/ai-ml) - Add fastapi-data (pre-commit + fastapi-redis + fastapi-rate-limit + fastapi-mlflow-tracing) covering remaining 4 fastapi orphans (ci/database/security/observability) — respects category uniqueness (one per category) - Add django-full (github-setup + development-container + django-docker + postgres + django-spectacular) covering django-spectacular orphan - Add mlops-sklearn-default (mlops-sklearn-starter + github-setup) as per PR #179 — cherry-picked profile and github-setup type extension for mlops-sklearn; pre-commit orphan is covered via fastapi-data (cannot co-locate ci category with github-setup in same profile) - Update templates.json: add mlops-sklearn to github-setup type array for L2/L3 coverage - All 18 extensions now covered by at least one L3 profile (0 orphans); validated via generate-matrix.py --layer validate-profiles and validate-registry.py Fixes orphan coverage noted in W4-1 and §3 (9/18 extensions orphan: cors, redis, sqlalchemy, otel, rate-limit, mlflow-tracing, rag-pgvector, django-spectacular, pre-commit)
|
Warning Review limit reached
Next review available in: 42 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
Fixes orphan extension coverage (W4-1, §3). 9/18 extensions had no L3 profile (
cors, redis, sqlalchemy, otel, rate-limit, mlflow-tracing, rag-pgvector, django-spectacular, pre-commit). Adds 4 curated profiles covering 100% of extensions plusmlops-sklearntemplate (PR #179).Changes
fastapi-starter+github-setup+fastapi-cors(security) +fastapi-sqlalchemy(database) +fastapi-opentelemetry(observability) +fastapi-rag-pgvector(ai-ml) — 5 distinct categories, allfastapi-backendcompatiblefastapi-starter+pre-commit(ci) +fastapi-redis(database) +fastapi-rate-limit(security) +fastapi-mlflow-tracing(observability) — respects one-per-category, covers remaining fastapi orphans andpre-commit(cannot co-locategithub-setup+pre-commit— samecicategory)django-api+github-setup+development-container+django-docker+postgres+django-spectacular— coversdjango-spectacularorphan (observability,django-backendonly)mlops-sklearn-starter+github-setup— cherry-picked from PR ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179 (8e581e6); will be deduplicated when ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179 merges. Note: task suggestedall-github-setup + all-pre-commitbut they sharecicategory — invalid perassert_profile_valid(category uniqueness).pre-commitorphan is already covered viafastapi-data.mlops-sklearntogithub-setuptypearray for L2/L3 coverage (as per ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179)Category / Type Validation
Each profile validated via
assert_profile_valid:categoryper profiletypeincludes the template's typeincompatibleWithrespected (none currently onmain;fastapi-opentelemetryvsfastapi-mlflow-tracingare kept in separate profiles to remain compatible if ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179'sincompatibleWithlands)How Has This Been Tested?
python scripts/ci/validate-registry.py— ✅ registry ok (6 templates, 18 extensions)python scripts/ci/generate-matrix.py --layer validate-profiles— ✅ 13 profiles validated (9 → 13)0orphans (was 9) —covered = 18/18python scripts/ci/generate-matrix.py --layer profiles— generates valid L3 matrixType of Change
Checklist
urlpoints to correct path onmainslugglobally uniquetypeis array where multi-type