-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
bumping common #14902
bumping common #14902
Conversation
I see you updated files related to
|
4686dfb
to
dc5c142
Compare
AER Report: CI Coreaer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , lint , Core Tests (go_core_tests) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , SonarQube Scan , Flakey Test Detection 1. Test failures and panics encountered:Error: Encountered test failures.[Run tests]Source of Error:
Suggested fix: Review the individual test failures to identify and fix the underlying issues causing the tests to fail. Error: Received unexpected error: error running module: error while executing at wasm backtrace:[Print Filtered Test Results]Source of Error:
Suggested fix: Investigate the WASM module and ensure that all pointers are properly initialized and handled to avoid nil dereferences. Error: Not equal: expected: "errored" actual : "completed"[Print Filtered Test Results]Source of Error:
Suggested fix: Review the workflow engine's error handling logic to ensure that steps are correctly marked as "errored" when an error occurs. Error: panic: interface conversion: values.Value is nil, not *values.Map [recovered][Print Filtered Test Results]Source of Error:
|
Quality Gate passedIssues Measures |
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.
This file will not exist in core anymore, the labeler was moved to common. Would you please update the labelers in common to reflect this?
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.
good catch, thanks
no longer needed; #14867 merged |
Requires: