docs: refresh public README and Pages navigation - #908
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughLineageWeave에 정적 공개 랜딩 페이지를 추가했습니다. 게시 스크립트는 랜딩 페이지를 검증하고 Changes공개 랜딩 페이지와 프로젝트 설명
랜딩 페이지 게시 및 검증
CI 트리거와 실행 환경 검증
Estimated code review effort: 3 (Moderate) | ~25 minutes Merge Risk: ⚪ Minimal · up to This refresh updates public documentation, Pages navigation, and explicit hosted-runner declarations without supplied evidence of a concrete correctness, security, availability, or deployment risk; no actionable merge-blocking risk remains after normal checks and review. Sequence Diagram(s)sequenceDiagram
participant GitHub Actions
participant publish_site
participant landing_source
participant output_index_html
GitHub Actions->>publish_site: 게시 작업 실행
publish_site->>landing_source: docs/index.html 검증
publish_site->>output_index_html: 랜딩 페이지 복사
output_index_html-->>GitHub Actions: 게시 결과 생성
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 4 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Signed-off-by: Codex <codex@localhost> Commit-Message-Assisted-by: Claude (via Claude Code)
Preserve the branch's explicit ubuntu-24.04 runner declarations while adopting protected main #923's docs-only Tests trigger filters.
Adopt #924's restored documentation-contract test triggering while preserving this branch's explicit ubuntu-24.04 hosted-runner declarations and public-surface delta.
seonghobae
left a comment
There was a problem hiding this comment.
Fresh protected-main review found a real convergence conflict, not a reason to close this PR. #908 modifies the same three repository-owned workflows that protected #931 later changed for Draft admission. GitHub now reports the PR non-mergeable. The valid #908 deltas are explicit ubuntu-24.04 runner pinning plus public-landing publication paths/tests; the valid protected deltas are ready_for_review/converted_to_draft handling, same-PR cancellation, and draft == false job admission. Neither side may be dropped. Repair must reconstruct the workflows with both contracts, preserve the public README/Pages/test delta, and retain protected tests/test_tests_workflow_contract.py. Pre-#931 hosted evidence does not transfer.
Merge the protected #931 workflow/contract blobs into the public-surface branch without dropping the branch's structural hosted-runner regression. This intentionally leaves the runner contract RED (`ubuntu-latest` vs required explicit `ubuntu-24.04`) so the follow-up can combine both valid contracts instead of choosing one side of the conflict.
Outcome
Refresh the public repository/Pages surface around LineageWeave's actual product responsibility while keeping evidence, license, and canonical-owner boundaries explicit. The root README/public landing becomes product-first, operator detail remains in its authoritative docs, publication fails closed when the source landing is absent, and repository-owned hosted jobs are pinned to an explicit supported image.
This PR does not introduce a release, customer, production deployment, benchmark superiority, certification claim, psychometric/statistical ownership, or model/provider routing implementation. TEPP/fast-mlsirm and contextual-orchestrator remain canonical owners.
Protected-main conflict repair
Fresh review
5116251331found a real convergence conflict after protected #931: #908 and currentmainboth modify Tests, PROV-O, and Ontology Pages workflows for different valid reasons. #908 needs explicitubuntu-24.04runners plus public-landing publication paths/tests; #931 needsready_for_review/converted_to_draft, same-PR cancellation, anddraft == falseadmission. Neither contract may be dropped.Realistic RED
01895d1b9cd37e8c625e28daadedf2a6a395840efirst merged the exact protected #931 workflow/contract blobs while retaining #908's structural hosted-runner regression. That deliberately made the branch'stests/test_github_hosted_runner_contract.pyfail because the protected workflows still declaredubuntu-latest.Causal convergence then combined both contracts:
53cbd8be9507c8a3bfb4f3bc249945beca7afc2b: Tests keeps fix(ci): defer repository-local jobs while PRs are Draft #931 Draft admission and pins both repository-owned jobs toubuntu-24.04;5e085c389731ab366210492a94f3aa7a3e1fdf78: PROV-O keeps fix(ci): defer repository-local jobs while PRs are Draft #931 Draft admission and pins its job toubuntu-24.04;00e90e03ae1afb7f13ae843dd578694d0f72b325: Ontology Pages keeps fix(ci): defer repository-local jobs while PRs are Draft #931 event/cancellation/Draft admission, restores docs: refresh public README and Pages navigation #908 public-landing paths/tests, and pins validate/publish toubuntu-24.04.The branch now contains protected
main@83eba56149eb802cd63642c507c324c9976ec78ewith no history rewrite and no gate weakening. The public README/Pages/test delta is preserved.Commercial-license prerequisite
Issue #910 remains the policy blocker for public-surface integration: the reachable
psycopg2-binaryLGPL-family path must be removed/replaced rather than hidden by attribution or a license exception. PR #911 is the active pg8000 replacement lane and is now Ready on its unchanged exact head for current-head validation; #908 must not land before that prerequisite is normally merged and the protected line is re-verified license-clean.Exact live boundary
main@83eba56149eb802cd63642c507c324c9976ec78e00e90e03ae1afb7f13ae843dd578694d0f72b325behind_by=033902990145and PROV-O33902990140: completedskippedunder Draft admission33902990079: both exact-head jobs completedskipped; GitHub resolved their runner labels asubuntu-24.04without allocating a runner33902990095, CodeQL33902990162, Security33902990087: queued at this snapshotKeep Draft behind #911 and current-head governance. Do not transfer predecessor checks/reviews, weaken license inventory, or publish/release from this head.