Lớp vendored lên kit 2.17.0 — lối cho một hồ sơ nghỉ (giữ fork stale-scope) - #126
Merged
Merged
Conversation
…ột hồ sơ nghỉ Ba tệp của INIT-CI-COPY-LIST đổi giữa 2.16.0 và 2.17.0; sáu tệp còn lại đã trùng khít 2.17.0 nên không chạm. Hai tệp lib chép nguyên bản. Tệp cổng là bản fork của kho này (cụm phạm vi hẹp `feature_scope`, giữ qua năm lượt nâng kit từ 1.38.0) nên đi đường gộp BA CHIỀU — nền 26bf12fe, của-ta là bản fork, của-họ là kit main. Gộp sạch, 0 xung đột; diff (+64/+37/+21) bằng đúng diff 26bf12fe→main của kit, và `feature_scope` còn nguyên bốn chỗ. Thứ kho này nhận được, đo trên chính cây này (base origin/main~1): - TRƯỚC: cổng exit 1 — `normalize-text-vi` chặn merge vì bằng chứng cũ hoá, mã đổi sau lượt verify. Hồ sơ đã có chữ ký người (Phan Le Manh 27/08) và món nợ ghim lại này không trả được bằng cách chạy lại. - SAU khi nâng: exit 1, GIỐNG HỆT từng dòng — nâng thước không tự đổi phán quyết, và không sinh đợt ghim lại nào. - Chiều đỏ: thêm một dòng nghỉ đủ vế vào chính hồ sơ ấy → cổng in NOTE gọi tên người và lý do, exit 0, `clean`. Dòng thử đã hoàn tác: viết dòng nghỉ thật là quyết định của người, không phải của lượt nâng thước này. Dòng nghỉ chỉ ăn trên hồ sơ đã có chữ ký người; thiếu bất kỳ vế nào (người · lý do · thời điểm) thì cổng nói ra và tiếp tục chấm như hồ sơ đang sống. Chữ ký không bị sửa — nó ở lại làm sử liệu. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Owner
Author
|
Về ô đỏ «Acceptance Gate» của CI — nó có trước PR này, và nó chính là thứ PR này mở lối. CI chạy Tái dựng đúng lệnh ấy tại chỗ, trên nhánh này:
Dòng thử đã hoàn tác. Cả hai lớp nợ của hồ sơ ấy — bằng chứng cũ hoá, và làn ghim lại kiểu cũ — đều nằm sau cùng một lối miễn, nên một dòng nghỉ đóng cả hai. Năm ô CI còn lại xanh. Việc còn lại là dòng nghỉ, và đó là quyết định của anh — lệnh ở phần cuối mô tả PR. |
Một dòng sổ, không sửa gì khác. Chữ ký Cổng Bằng chứng (Phan Le Manh 27/08) và toàn bộ bằng chứng ở lại nguyên vẹn làm sử liệu — hồ sơ chỉ thôi bị chấm ở phần còn lại của lưới. Lý do là quyết định của người: vật đã đi tiếp sau chữ ký, lời hứa của hồ sơ không còn ứng với mã hôm nay, nên món nợ ghim lại không còn nghĩa để trả. Hệ quả đo được, đúng lệnh CI (`--base origin/main`): trước 1 vi phạm — merge bị chặn; sau `pre-merge-check: clean`, kèm NOTE gọi tên người và lý do. Hai lớp nợ của hồ sơ này — bằng chứng cũ hoá và làn ghim lại kiểu cũ — cùng nằm sau một lối miễn nên một dòng đóng cả hai. Lối quay lại nếu quyết sai: một dòng `supersedes` trỏ id này, hồ sơ sống lại. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Kho này đang trả gì hôm nay
Cổng trước-merge của OneFlow đang chặn:
Hồ sơ ấy đã có chữ ký người (Phan Le Manh, 27/08). Mã đổi sau lượt verify, nên món nợ «ghim lại» này không trả được bằng cách chạy lại — và luật làn eval nói pin chưa chứng là nợ ở mọi lượt chạy. Trước 2.17.0 chỉ còn hai lựa chọn: kẹt vĩnh viễn, hoặc sửa chữ ký. Chữ ký là sử liệu nên không được sửa.
Thứ nhận được
Một lời hứa đã ký mà món nợ không bao giờ trả được nay có lối ra: một dòng sổ có người, lý do và thời điểm. Hồ sơ thôi bị chấm ở phần còn lại của lưới; chữ ký ở lại nguyên vẹn làm sử liệu.
Dòng nghỉ chỉ ăn trên hồ sơ đã có chữ ký người. Thiếu bất kỳ vế nào thì cổng nói ra và tiếp tục chấm như hồ sơ đang sống — không có đường tắt im lặng.
Phạm vi và cách nâng
Ba tệp của
INIT-CI-COPY-LISTđổi giữa 2.16.0 và 2.17.0; sáu tệp còn lại đã trùng khít 2.17.0 nên không chạm.Tệp cổng là bản fork của kho này — cụm phạm vi hẹp
feature_scope, giữ qua năm lượt nâng kit từ 1.38.0 — nên đi đường gộp ba chiều (nền26bf12fe· của-ta là fork · của-họ là kitmain). Gộp sạch, 0 xung đột;feature_scopecòn nguyên bốn chỗ. Hai tệplib/chép nguyên bản. Tổng diff bằng đúng diff26bf12fe→maincủa kit: +64 / +37 / +21.Đã đo trên chính cây này (base
origin/main~1)normalize-text-vicũ hoá)pre-merge-check: clean, cổng in NOTE gọi tên người + lý doViệc còn lại là của anh
PR này không viết dòng nghỉ cho
normalize-text-vi— dòng thử đã hoàn tác. Cho một hồ sơ nghỉ là quyết định của người, không phải của lượt nâng thước. Sau khi PR này vàomain, một lệnh mở được cổng:🤖 Generated with Claude Code