-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add boxed variant comparisons #542
Conversation
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (36.32%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #542 +/- ##
==========================================
- Coverage 80.49% 80.08% -0.42%
==========================================
Files 95 95
Lines 20266 20465 +199
Branches 20266 20465 +199
==========================================
+ Hits 16314 16390 +76
- Misses 3520 3640 +120
- Partials 432 435 +3 ☔ View full report in Codecov by Sentry. |
Conformance comparison report
Number passing in both: 5511 Number failing in both: 880 Number passing in Base (cc421cf) but now fail: 0 Number failing in Base (cc421cf) but now pass: 38 The following test(s) were previously failing but now pass. Before merging, confirm they are intended to pass: Click here to see
|
7f79f58
to
6535718
Compare
There are some lingering warnings of the form They are fixed in #544 : for example: https://github.com/partiql/partiql-lang-rust/pull/544/files#diff-b95d0de35899342f30d2531c8cd3728a958d846d163f0c154c204f7aeb8979ceL112 |
95a28d7
to
68f01d4
Compare
This PR builds on #540 and adds comparisons and comparison tests.
There are still
todo
s and some additional test failures that will be addressed by future PRs that add functionality.To see the ultimate end-point of this integration, refer to #536 and note the test coverage and conformance test results.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.