shazhou/united-workforce
8
0
Fork 0
Code Issues 8 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: add missing workflow destructure in current-role test #57

Merged
xiaoju merged 1 commits from fix/56-ts-compile-error into main 2026-06-04 11:00:26 +00:00
Conversation 0 Commits 1 Files Changed 1
+1 -1
xiaoju commented 2026-06-04 10:56:53 +00:00
Owner
Copy Link
Copy Source

One-liner: destructure workflow from cmdThreadStart return value so createMarker compiles.

Also found 6 test failures due to test isolation (store data leaking between tests) — separate issue.

Fixes #56

One-liner: destructure `workflow` from `cmdThreadStart` return value so `createMarker` compiles. Also found 6 test failures due to test isolation (store data leaking between tests) — separate issue. Fixes #56
xiaoju added 1 commit 2026-06-04 10:56:54 +00:00
fix: add missing workflow destructure in current-role test
CI / check (pull_request) Failing after 1m37s
Details
9f5891169e 
The createMarker call used shorthand 'workflow' but the variable
was not destructured from cmdThreadStart.

Fixes #56
xiaoju merged commit 5fe492c011 into main 2026-06-04 11:00:26 +00:00
xiaoju deleted branch fix/56-ts-compile-error 2026-06-04 11:00:26 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/united-workforce#57
Delete Branch "fix/56-ts-compile-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?