-
Notifications
You must be signed in to change notification settings - Fork 1
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
Uninstall Workflow #29
Conversation
I'll have a looky |
I can't see anything inherently wrong with the modifications, but do make sure to test (and not do a crowdstrike). |
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.
Looks good, pls test (including edge cases).
I tested it on W10 (w/ OF in the default sourcemod folder) last night, but I'll check it on my Debian install and my Steam Deck tonight. |
Tested on my windows install. Need to rebase from develop since #30 was merged. |
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.
looks good
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.
compiles fine
This PR adds an
uninstall
CLI sub-command and wizard option for uninstalling the target game. Before deleting the sourcemod mod folder, it will check if all of the following conditions are met;RunnerContext.current_version.is_some()
)crate::helper::is_game_running
wheremod_directory
is fromRunnerContext.get_mod_location()
)If any of those conditions are not met, an error will be returned and will not allowed the user to uninstall the game (if it is even installed).