Skip to content

Conversation

@sue-lim
Copy link
Contributor

@sue-lim sue-lim commented Jun 15, 2025

Maintenance: Complete Docs, Tests, and Finalized CLI for Currency Script

[MAINTENANCE] Currency Script | Expand Test Coverage, Add CLI (main.py), and Improve Docs #433

Summary

This PR performs essential maintenance tasks to ensure the currency converter script is documented, testable, and complete for use via a CLI entry point (main.py).

Description

This update focuses on project hygiene and completeness, without modifying or refactoring any core logic. The goal was to tidy up the structure, add missing elements, and prepare the project for practical use or future extension.

The changes are as follows:

  • Completed docstrings across all modules:

    • api_handler.py
    • currencies.py
    • converter.py
    • All test_*.py files
  • Added unit tests for:

    • Exchange rate API logic
    • Currency conversion logic
    • Currency utility validation
  • Finalized main.py to act as a CLI interface for converting currencies interactively

  • Updated requirements.txt for dependency installation (only requests)

  • Updated a simple README.txt to explain setup and usage

  • Updated setup.py for packaging (basic scaffolding only)

Checks

in the repository

  • Made no changes that degrades the functioning of the repository
  • Gave each commit a better title (unlike updated README.md)

in the PR

  • Followed the format of the pull_request_template
  • Made the Pull Request in a small level (for the creator's wellfare)
  • N/A Tested the changes you made

Thank You,
Sue

@sue-lim
Copy link
Contributor Author

sue-lim commented Jun 15, 2025

@DhanushNehru please approve :)
This is the last of the many.

@DhanushNehru DhanushNehru merged commit ca01f11 into wasmerio:main Jun 16, 2025
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