Follow-up to #6661: Reboot sync: a watch fault after activity creation must report the import still running
Maintainer thường phản hồi trong vòng 1 ngày
Một pull request liên quan đã được merge.
- #6661 của @rbuergi — đã merge
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 40/100
Hướng nghiên cứu
Start from the reboot sync code that calls onActivityCreated and the catch around the terminal-status watch, where StillRunning is set to null. Trace how Sequence decides to start the next import, and read the pull request #6661 discussion for the intended behaviour. Write a failing test where the watch faults after activity creation and expect StillRunning to be returned, then make the fix. Done when that test passes and pre-creation failures still let the sequence continue.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Follow-up to #6661 (review findings deferred from that pull request; Refs #6661).
- If the terminal-status watch faults after
onActivityCreated, the catch reportsStillRunning = nullandSequencestarts the next import while the first may still write. Keep the created activity path and return it as still running for every post-creation watch failure; only pre-creation failures may let the sequence continue.
These are automated-review findings, assessed from the diff and not reproduced; each needs a failing test before its fix. Deferred from the PR because CI is saturated and none of them is a security hole, data loss or a broken build/test.
- Ngôn ngữ chính
- C#
- Star
- 12
- Fork
- 5
- Merge trung bình
- 5 giờ 51 phút
- Pull request đã merge (30 ngày)
- 970
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Systemorph/MeshWeaver
-
bug sev:M
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
Systemorph/MeshWeaver#6896 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug sev:M
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 12/100
Systemorph/MeshWeaver#6857 · 11 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug sev:M
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
Systemorph/MeshWeaver#6856 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug cli-publish-failure feature:release-promotion sev:M
Độ khó 3/5 1-3 giờ Mức phù hợp với người mới 12/100
Systemorph/MeshWeaver#6845 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug feedback sev:H
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 22/100
Systemorph/MeshWeaver#6827 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của Systemorph/MeshWeaver
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
pyrevitlabs/pyRevit#3792 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 60/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area: design documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
eyalgolan/statecraft-israel#883 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 3 ngày
-
area/indexing bug package/core triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày