Skip to content

Update README.md#192

Open
lucasfonte21 wants to merge 3 commits into
ufosc:mainfrom
lucasfonte21:main
Open

Update README.md#192
lucasfonte21 wants to merge 3 commits into
ufosc:mainfrom
lucasfonte21:main

Conversation

@lucasfonte21
Copy link
Copy Markdown

Issue #191 Made the change described in the thread.

Issue ufosc#191 Made the change described in the thread.
Comment thread README.md Outdated
- install necessary python packages using pip:
- ```pip install -r requirements.txt```
- run using python monopoly.py
- run using "python -m monopoly_directory.monopoly" from TERMINALMONOPOLY (root folder)
Copy link
Copy Markdown

@kevin8999 kevin8999 Feb 16, 2026

Choose a reason for hiding this comment

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

I think it would be best to replace the quotation marks (") with backticks (`). This makes it obvious that it is a command.

Before (")

  • run using "python -m monopoly_directory.monopoly" from TERMINALMONOPOLY (root folder)

After (`)

  • run using python -m monopoly_directory.monopoly from TERMINALMONOPOLY/ (root folder)

Fixed it to make it look more like a command.
Updated the logic in the calibrate_screen function so inputting an invalid input makes you go again instead of picking a default color.
@lucasfonte21
Copy link
Copy Markdown
Author

Also fixed issue #190

@gitdownup
Copy link
Copy Markdown

gitdownup commented Feb 17, 2026

@lucasfonte21 I think it might be best to just add monopoly.py to the root of the TERMINALMONOPOLY/ directory. Then, one just needs to run python monopoly.py or something equivalent. I have done this as part of a PR that I could post to this repo.

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.

3 participants