-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat: Add list.product #24382
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: main
Are you sure you want to change the base?
feat: Add list.product #24382
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #24382 +/- ##
========================================
Coverage 81.77% 81.77%
========================================
Files 1681 1682 +1
Lines 228116 228353 +237
Branches 2923 2923
========================================
+ Hits 186536 186743 +207
- Misses 40847 40877 +30
Partials 733 733 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I would rather have this be implemented through #22650. I personally think having the syntax sugar is fine, but I think the kernels are a bit overkill. |
close #12862