-
Notifications
You must be signed in to change notification settings - Fork 1.9k
core/services/workflows/types: add WorkflowID.String() #20703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
👋 jmank88, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
I see you updated files related to
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔴 Test Results: Unrelated Failure
Affected failures:
- Workflow Run: Integration Tests
What Broke
This failure appears to be unrelated to the changes in this PR. The CI failure is not caused by the changes introduced in this PR. The failure is due to an external, unrelated issue within the CI environment or the existing codebase. The PR only adds a String() method to the WorkflowID type, and a code search confirms this new method is not being called anywhere in the codebase. The CI failure logs are generic, indicating 'Process completed with exit code 1.' without any specific error messages or stack traces that could be linked to the PR's changes.
Autofix Options
You can use our MCP server to get AI assistance with debugging and fixing these failures.
- Use MCP in your IDE to debug the issue. Try
Help me fix CI failures from 0gQip4KHto get started.




These log as raw bytes otherwise, which is spammy and unreadable.