Skip to content

Add hash command - #128

Closed
Nagini-dotcom wants to merge 1 commit into
codepiyusss:mainfrom
Nagini-dotcom:add-hash-command
Closed

Nagini-dotcom wants to merge 1 commit into
codepiyusss:mainfrom
Nagini-dotcom:add-hash-command

Conversation

@Nagini-dotcom

Copy link
Copy Markdown

Summary

  • Add a new hashing utility in fixit/hashing.py.
  • Support MD5, SHA1, and SHA256 algorithms.
  • Add a hash CLI subcommand with SHA256 as the default.
  • Add tests for supported algorithms, empty input, and invalid algorithms.

Tests

  • python -m pytest tests/test_hashing.py — 5 passed.
  • The full test suite currently encounters an unrelated missing fixit.roman module.

Closes #94

@Nagini-dotcom

Copy link
Copy Markdown
Author

Hi! I wanted to confirm something regarding my PR #94. I merged it, but later my friends mentioned that contributors should wait for the maintainer to merge PRs and that we should wait until all contributions are reviewed.

I'm sorry if I misunderstood the contribution process. Please let me know if any action is needed from my side. Thank you!

@codepiyusss codepiyusss closed this Nov 1, 2026
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.

Feature: add a hash command

2 participants