Skip to content

Conversation

@IceWizard7
Copy link
Contributor

Type annotations

  • Add type annotations to functions (e.g. def run(args: typing.Any) -> None:)
  • Add '_' in front of unused variables (e.g. def run(_args: typing.Any) -> None:)

No overiding built-ins

  • Use command_help: str = ... instead of help

Unnecessary imports

  • Remove unnecessary imports (e.g. os)

New commands

  • Add rename.py command

Code style

  • Add newlines, consistent usage of tabs & spaces, etc.

@IceWizard7
Copy link
Contributor Author

plz add me as contributer in the readme lol

@jacklak-redstone
Copy link
Contributor

LGTM

@jacklak-redstone
Copy link
Contributor

Hold on. The rename command will crash when only given 1 arg. Also, os.rename is a much cleaner way to do this.

@IceWizard7
Copy link
Contributor Author

Uhh thanks for noticing. Ima look into it in a bit.

@IceWizard7
Copy link
Contributor Author

IceWizard7 commented Nov 23, 2025

It's now using os.rename, thanks!

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