Skip to content

[tool] octal_to_decimal#224

Open
selvakanthanjagavan-byte wants to merge 1 commit into
abduznik:mainfrom
selvakanthanjagavan-byte:tool-octal-to-decimal
Open

[tool] octal_to_decimal#224
selvakanthanjagavan-byte wants to merge 1 commit into
abduznik:mainfrom
selvakanthanjagavan-byte:tool-octal-to-decimal

Conversation

@selvakanthanjagavan-byte

Copy link
Copy Markdown

Checklist

  • File is in tools/ directory
  • File has header comments (# tool, # description, # author, # example)
  • File has a run() function that returns a string
  • Uses stdlib only (no external dependencies)
  • Tested locally with python bitbox.py octal_to_decimal 77

Closes #207

@abduznik

Copy link
Copy Markdown
Owner

Hey! Thanks for the contribution. A few things before merging:

  1. No error handling — If someone runs or , it will crash. Please add error handling for invalid octal input.
  2. Format — Use header format instead of docstring (see other tools in for reference).
  3. Missing trailing newline — Add a newline at end of file.

The core logic looks correct. Once these are fixed, good to go!

@abduznik abduznik mentioned this pull request Jul 21, 2026
5 tasks
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.

[tool] octal_to_decimal

3 participants