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

`BlankLabelError` mistakenly use `rowNumbers` (plural) despite returning a single row number #1710

Open

1 task

pierrecamilleri opened this issue Nov 22, 2024 · 0 comments

Collaborator

pierrecamilleri commented Nov 22, 2024

The BlankLabelError has a different API. Instead of rowNumber it has rowNumbers . Grepping the code I see that that's the only reference to rowNumbers and the docstring even suggest that it should be a value instead of a list. Is this an expected behavior of a bug?

check that the code is aligned with documentation and that there is no edge case that needs to return several rowNumbers.

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