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.

Sign up for GitHub

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

Jump to bottom

feat: fix function lookup in coverage tool #744

Merged

jacques-n merged 1 commit into substrait-io:main from srikrishnak:fix-function-lookup

Nov 18, 2024

Contributor

srikrishnak commented Nov 18, 2024

No description provided.


          feat: fix function lookup in coverage tool

67eb1c8

srikrishnak requested review from jacques-n, cpcloud, westonpace, EpsilonPrime and vbarua as code owners

November 18, 2024 14:27

srikrishnak mentioned this pull request

feat: add bft function testcases in new substrait testfile format #738

Merged

EpsilonPrime approved these changes

View reviewed changes

Member

EpsilonPrime left a comment

We should add a workflow to run the tests but that can be done in a separate PR.

jacques-n approved these changes

View reviewed changes

jacques-n merged commit 3d2ff77 into substrait-io:main

13 checks passed

srikrishnak deleted the fix-function-lookup branch

November 19, 2024 04:27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jacques-n jacques-n approved these changes

EpsilonPrime EpsilonPrime approved these changes

cpcloud Awaiting requested review from cpcloud cpcloud is a code owner

westonpace Awaiting requested review from westonpace westonpace is a code owner

vbarua Awaiting requested review from vbarua vbarua is a code owner

Labels

None yet