Skip to content

chore: remove stale TODOs and commented-out code #408

Open

Description

@refeed

Problem

src/tirith/cli.py carries a commented-out --input-type check under # TODO: move to core, and src/tirith/utils.py has a # TODO: LOG directly above a line that already logs.

What to change

Delete those two spots only. Leave every other TODO; they describe real open work.

Done when

The diff removes only comments and pytest tests/ passes.


Comment to claim this issue and wait to be assigned. Unclaimed or inactive for 3 days, it returns to the pool. Run pytest tests/ and black --line-length 120 before opening the PR.

Activity

  1. krishna2707 commented on Oct 14, 2026

    @krishna2707

    Hi! I'd like to work on this issue. Could you please assign it to me? I'll remove only the two specified comment sections, then run pytest tests/ and black --line-length 120.

  2. refeed commented on Oct 21, 2026

    @refeed
    MemberAuthor

    Hi @krishna2707 , sorry for the late response, just assigned :)

  3. Ankit-cs commented on Oct 21, 2026

    @Ankit-cs
    Contributor

    hey @krishna2707 since i has an active pr for the above issue can you look for the another once?

  4. refeed commented on Oct 31, 2026

    @refeed
    MemberAuthor

    Hi Ankit, I just merged your PR. However anybody can still open PR to this because I think there are still some parts of the code that still have stale comments

  5. Ankit-cs commented on Oct 31, 2026

    @Ankit-cs
    Contributor

    ok @refeed i can just look for that and remove other also and create another pr for the same title and description.

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions