eranheres / ultra_type

0 stars 0 forks source link

Sweep: modify the application main loop and states #33

Open eranheres opened 11 months ago

eranheres commented 11 months ago

Main menu should be:

Change user - this option should change the user. The default user name is "default". Select language - this option should change the language. Default language is "English" Practice - enter a practice mode Show statistics - show the statistics Exit Apply unit testing Modify README file accordingly.

Checklist - [X] Modify `ultra_type/model.py` ✓ https://github.com/eranheres/ultra_type/commit/d4e8324062104dc1aaca2c7699defffa4a0313a7 - [X] Modify `ultra_type/controller.py` ✓ https://github.com/eranheres/ultra_type/commit/7e5cb1924b25ce837fa909c59e2c8d525564d3cd - [X] Modify `ultra_type/main.py` ✓ https://github.com/eranheres/ultra_type/commit/74ea728e9a0bec2c39618c658c72b568bca6ad43 - [X] Create `tests/test_user.py` ✓ https://github.com/eranheres/ultra_type/commit/c73cbfef4a9b4878c65900bc893693c0ecd1524a - [X] Modify `README.md` ✓ https://github.com/eranheres/ultra_type/commit/972e13efb46caed86425b73be3f2765d1cce8919 ![Flowchart](https://raw.githubusercontent.com/eranheres/ultra_type/sweep/assets/ec26a08e2f53819e38448c6fcb4e53586b91a560b0a6065227cc5095581c3dba_33_flowchart.svg)
sweep-mine[bot] commented 11 months ago

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

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

Actions (click)

Sandbox Execution X

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

Sandbox logs for af2df64
poetry run pylint -E **/*.py 1/1 ❌ (`2`)
************* Module tests.test_model
tests/test_model.py:5:0: E0611: No name 'Model' in module 'ultra_type.model' (no-name-in-module)
************* Module ultra_type.controller
ultra_type/controller.py:1:0: E0611: No name 'Model' in module 'ultra_type.model' (no-name-in-module)
************* Module ultra_type.main
ultra_type/main.py:3:0: E0611: No name 'Model' in module 'ultra_type.model' (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/af2df64cd6e85f247912304705b79b60c1842386/README.md#L1-L20 https://github.com/eranheres/ultra_type/blob/af2df64cd6e85f247912304705b79b60c1842386/ultra_type/controller.py#L1-L33 https://github.com/eranheres/ultra_type/blob/af2df64cd6e85f247912304705b79b60c1842386/tests/test_language.py#L1-L79 https://github.com/eranheres/ultra_type/blob/af2df64cd6e85f247912304705b79b60c1842386/tests/test_model.py#L1-L142 https://github.com/eranheres/ultra_type/blob/af2df64cd6e85f247912304705b79b60c1842386/ultra_type/__init__.py#L1-L-1

Step 2: ⌨️ Coding


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/user-language-menu-update.


🎉 Latest improvements to Sweep:


💡 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