-
-
Notifications
You must be signed in to change notification settings - Fork 999
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
fix(module: tree): Fix search crash issue after Tree custom SearchExpression #3274
fix(module: tree): Fix search crash issue after Tree custom SearchExpression #3274
Conversation
@dotnet-policy-service agree |
Thanks for contribution @ruyisee , please chekout the review commonts. |
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.
LGTM
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3274 +/- ##
==========================================
+ Coverage 45.27% 46.70% +1.43%
==========================================
Files 557 557
Lines 26725 26731 +6
Branches 266 266
==========================================
+ Hits 12100 12485 +385
+ Misses 14585 14206 -379
Partials 40 40
☔ View full report in Codecov by Sentry. |
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge