Skip to content

Comments

commit for the Python WPM Typing program#16

Open
SamuelCuerrier01 wants to merge 1 commit intoNeuroByte-Society:mainfrom
SamuelCuerrier01:main
Open

commit for the Python WPM Typing program#16
SamuelCuerrier01 wants to merge 1 commit intoNeuroByte-Society:mainfrom
SamuelCuerrier01:main

Conversation

@SamuelCuerrier01
Copy link

This is my version of the WPM typing program pretty simple

Copy link
Contributor

@ghostdeveloper404 ghostdeveloper404 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks — nice simple typing tester! Two quick requests before approving:

  1. Please add a README.md (per the "Add README.md" issue) that explains:

    • What the project does
    • How to run it
    • Example input/output
  2. Small code suggestions:

    • Use time.perf_counter() for timing
    • Count extra typed characters as errors
    • Consider WPM = (chars / 5) / minutes or document your method
    • Wrap logic in main() and add if __name__ == "__main__":

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