Skip to content
This repository was archived by the owner on Aug 6, 2024. It is now read-only.

Various changes (see description) #147

Draft
silvncr wants to merge 14 commits into
base: master
Choose a base branch
from
Draft

Various changes (see description) #147

silvncr wants to merge 14 commits into from

Conversation

@silvncr
Copy link
Member

@silvncr silvncr commented Apr 15, 2023 •
edited
Loading

Ease of use

  • Replace input() with msvcrt.getche(), lowering the key presses per input from 2 to 1

File system

  • Move code into separate files relating to its purpose

Code quality

  • Use more classes for repetitive tasks (in-game locations, etc.)

@silvncr silvncr linked an issue Apr 15, 2023 that may be closed by this pull request
Replace input() with msvcrt/getch functions #144
Open
@silvncr silvncr self-assigned this Apr 15, 2023
@silvncr silvncr changed the title Huge rewrite Various changes (see description) Apr 17, 2023
@silvncr silvncr added bug Something isn't working enhancement New feature or request dependencies Pull requests that update a dependency file maintenance GitHub documentation or issues/PRs labels Apr 20, 2023
@sourcery-ai sourcery-ai bot mentioned this pull request Oct 19, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@silvncr silvncr

Labels

bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request maintenance GitHub documentation or issues/PRs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Replace input() with msvcrt/getch functions

3 participants

@silvncr @nasserkessas