-
Notifications
You must be signed in to change notification settings - Fork 3.7k
enhance: [2.6] ValidateAnalyzer return ValidateAnalyzerResponse instead common.Status #46294
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
base: 2.6
Are you sure you want to change the base?
Conversation
Signed-off-by: aoiasd <[email protected]>
|
[ci-v2-notice]
To rerun ci-v2 checks, comment with:
If you have any questions or requests, please contact @zhikunyao. |
|
[INFO] PR Label Summary by Default
[WARNING] Milestone not set
You can set milestone by commenting: Use /refresh-label to update related check and label manually |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 2.6 #46294 +/- ##
===========================================
+ Coverage 73.54% 82.76% +9.22%
===========================================
Files 1362 521 -841
Lines 212572 80978 -131594
===========================================
- Hits 156330 67022 -89308
+ Misses 48809 13956 -34853
+ Partials 7433 0 -7433
🚀 New features to boost your workflow:
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aoiasd, zhengbuqian The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
[INFO] PR Label Summary by Default
Use /refresh-label to update related check and label manually |
Prepare for return more info when validate analyzer.
relate: #43687
pr: #46292