-
Notifications
You must be signed in to change notification settings - Fork 13
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
General improvements #1
Conversation
Alex-1000
commented
Jun 4, 2022
- Sourcemod folder search now should work for Linux too
- Library imports have been edited to only import parts that are used in code
- Descriptions were written for functions
- Program asks for OS name only once now
- "message" and "message_quick" were merged into single function
- Some messages were merged together
- "message_yes_no" now has default answer option
i forgot to sort libraries in some files, standard libraries are usually put above program's own |
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.
Right now looks pretty good. Merging message_quick with message is also pretty good. We're planning on using the same functions to be able to actually show a GUI.
ans = input(msg + " (Y/N)>").lower() | ||
|
||
if ans == "y": | ||
def message_yes_no(msg:str, default = true): |
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.
Not sure I like this when people could accidentally put in an invalid answer, then the program just chooses it for them.
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.
fair enough
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.
I'd like to note that we're currently looking to use the registry.vdf file to get the sourcemods path on Linux.
LGTM, sorry for the wait. Even though some solutions are kind of dirty (like manually combing through possible paths for the sourcemods folder on Linux, which won't work for Flatpak, Snap, server installs using steamcmd...), we can improve on that after it's merged. |