Pull Request Overview

This pull request updates the configuration to enable flake8-todos support and revises TODO comments in the Python code to include a username.

• Adds flake8-todos ("TD") to the lint.select list in pyproject.toml.
• Introduces a new ignore rule ("TD003") in pyproject.toml and updates inline TODO comments in cpplint_clitest.py and cpplint.py to include a username placeholder.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

cpplint_clitest.py:66

• Replace 'unknown' with a valid username to meet the requirement that all TODO comments include the responsible individual's username.

        # TODO(unknown): Support scenario with multiple input names

cpplint.py:4248

• Replace 'unknown' with a valid username in the TODO comment so that code ownership is clear.

        # TODO(unknown): support alternate operators

cpplint.py:4770

• Replace 'unknown' with a valid username to properly attribute the TODO comment.

    # TODO(unknown): Err on if...else and do...while statements without braces;

cpplint.py:7460

• Replace 'unknown' with a valid username to comply with the convention for TODO comments.

        # TODO(unknown): Maybe make this have an exit code of 2 after all is done

What is the point of making these changes if they just going to silence/workaround it by making it all unknown? How does this provide a point of reference for the best context? To do this, we should correctly blame all the TODOs. (BTW, since we check all the flake8 rules we check with ruff too, I think we can go ahead with removing flake8 from pre-commit.)

(psssst: I hate to sound like a broken record, but I'd really appreciate a review for #353.)