Skip to content

Conversation

@VamatoHD
Copy link
Collaborator

@VamatoHD VamatoHD commented Dec 12, 2025

Following issue #75, this PR contains the start of its implementation. I'm aware that, even though I haven't implemented CSV yet, much has changed.

  1. Made printer.py which contains a class Printer that handles the printing. It is passed to almost every function in orchestrator.py so they can handle printing.
  2. New parameter last needed to remove the last comma in JSON output format.
  3. Removes delay after the last printing statement.

@kaifcodec If you have any suggestions, feel free to mention them.

To Do:

  • Implement CSV.
  • Change the output's indentation to 2 spaces instead of \t.
  • Remove duplicated logic in __main__.py.
  • Clean up code, especially variable naming.
  • Update README.md.
  • Testing.

@kaifcodec
Copy link
Owner

@VamatoHD I think everything looks good so far, including the Printer class. It feels reusable and could be handy later for different modes as well.
Take your time and keep going, your approach looks solid.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants