-
Notifications
You must be signed in to change notification settings - Fork 417
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 build warnings about missing aggregates #1368
Conversation
|
Codecov Report
@@ Coverage Diff @@
## main #1368 +/- ##
=======================================
Coverage 85.30% 85.30%
=======================================
Files 154 154
Lines 4374 4374
=======================================
Hits 3731 3731
Misses 643 643
|
@taekahn thanks for the fix. Could you please help fix the CLA issue? |
I'm not sure what i have to do. I clicked on the link in the bot message and signed the form 3 times. |
@taekahn Can you sign up for EasyCLA so that we can merge the PR ? |
As i said above, i clicked on the link. Registered. Signed the form several times. |
The commit is: Jeffrey Reynolds authored and taekahn committed on May 18 It could be that the missing CLA and id is about Jeffrey, not taekahn. |
Fixes # (issue)
Changes
Fixing a build warning about missing initialize for aggregate initialization.