New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Improve table detection in checkers #70

Open

ezio-melotti opened this issue Jul 30, 2023 · 0 comments

Labels

Collaborator

ezio-melotti commented Jul 30, 2023

In #65 I implemented a check for tables:

        if (stripped_line.startswith("|") and stripped_line.endswith("|") and
            stripped_line.count("|") >= 4 and "|" in match.group(0)):
            return  # we don't handle tables yet.

that is more robust than the one used in other checkers:

        if paragraph.count("|") > 4:
            return  # we don't handle tables yet.

This check should be factored out in a separate function, and used in the other checkers too.

ezio-melotti added the enhancement label

# for free to join this conversation on GitHub. Already have an account? # to comment