Sweep: modify get_language_code to have option selection using numbers

Here's the PR! https://github.com/eranheres/ultra_type/pull/31.

💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: b610a790cc)

Actions (click)

[ ] ↻ Restart Sweep

Sandbox Execution X

Here are the sandbox execution logs prior to making any changes:

Sandbox logs for 7d64cac

poetry run pylint -E **/*.py 1/1 ❌ (`2`)

************* Module ultra_type.controller
ultra_type/controller.py:2:0: E0611: No name 'View' in module 'ultra_type.view' (no-name-in-module)
************* Module ultra_type.main
ultra_type/main.py:2:0: E0611: No name 'View' in module 'ultra_type.view' (no-name-in-module)

Sandbox failed, so all sandbox checks will be disabled for this issue.

Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/eranheres/ultra_type/blob/7d64caccbe0d4a9ad1480594fae9885104794235/ultra_type/view.py#L1-L22 https://github.com/eranheres/ultra_type/blob/7d64caccbe0d4a9ad1480594fae9885104794235/ultra_type/language.py#L1-L17 https://github.com/eranheres/ultra_type/blob/7d64caccbe0d4a9ad1480594fae9885104794235/ultra_type/controller.py#L1-L32 https://github.com/eranheres/ultra_type/blob/7d64caccbe0d4a9ad1480594fae9885104794235/README.md#L1-L20

Step 2: ⌨️ Coding

[X] Modify ultra_type/view.py ✓ https://github.com/eranheres/ultra_type/commit/04201a4483bdda82e5a1cfdb56f3090869defadc

Modify ultra_type/view.py with contents:
• In the 'get_language_choice' method of the 'View' class, change the string in the 'addstr' method to "Choose a language: 1. English 2. Hebrew".
• Modify the return statement to convert the user's input to an integer and then map it to the corresponding language. If the user enters 1, return 'English'. If the user enters 2, return 'Hebrew'.
[X] Modify ultra_type/controller.py ✓ https://github.com/eranheres/ultra_type/commit/a2ee62806dd2acd4b01e3991b550472f0ff8a1a3

Modify ultra_type/controller.py with contents:
• In the 'practice' method of the 'Controller' class, update the 'language' variable assignment to handle the new number-based language selection. The 'get_language_choice' method now returns a string representing the language, so no changes are needed here.

Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/number-based-language-selection.

🎉 Latest improvements to Sweep:

Sweep now uses the rope library to refactor Python! Check out Large Language Models are Bad at Refactoring Code. To have Sweep refactor your code, try sweep: Refactor <your_file>.py!
Sweep finds and fixes bugs in your code by writing unit tests! Check out Having GPT-4 Iterate on Unit Tests like a Human.

💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. ^{Join Our Discord}

eranheres / ultra_type