-
Notifications
You must be signed in to change notification settings - Fork 722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generate: improve error message for missing game #1857
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if ret.game in AutoWorldRegister.world_types:
in line 469 and else
in line 493 can go now?
As a side note, the fuzzy matching is not very good:
:D |
I mean, half the words are there! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The last commit looks super confusing in the github view, but I think it's fine
--------- Co-authored-by: black-sliver <[email protected]>
--------- Co-authored-by: black-sliver <[email protected]>
--------- Co-authored-by: black-sliver <[email protected]>
What is this fixing or adding?
When using a world that doesn't exist, such as when missing an apworld, the user would just get a KeyError:
<name of world>
How was this tested?
Exception: No world found to handle game fAcToRiO. Did you mean 'Factorio' (100% sure)? Check your spelling or installation of that world.
If this makes graphical changes, please attach screenshots.