This repository has been archived by the owner on Dec 22, 2023. It is now read-only.
Remove useless comments #85
Comments
sveeke
referenced
this issue
Sep 6, 2019
- Changed variable when no serverbot.conf is available to be more clear. - Some minor text improvements. - Added basic configuration file parameter validation https://github.com/nozel-org/serverbot/issues/83. - Removed placeholders for feature login and outage. - Removed abundance of comments that weren't very useful https://github.com/nozel-org/serverbot/issues/85.
|
Fixed in nozel-org/serverbot@96dbb45. |
sveeke
added a commit
that referenced
this issue
Sep 24, 2019
This is a combined update for all `1.1.0-UNSTABLE` commits. - Changed suffixes from `ALPHA`, `BETA` and `RELEASE` to new suffix layout `UNSTABLE` and `STABLE`. - Added feature EOL #63. - Added basic configuration file parameter validation #83. - Added notification to user on serverbot upgrade #84. - Removed abundance of comments that weren't very useful #85. - Added error when method telegram is not configured #88. - Made all gather functions calls consistent by placing them inside relevant functions #89. - Added branch variable in serverbot #90. - Removed argument shortcuts because of conflicting names #92. - Removed useless breaks in feature updates method cli #93. - Improved validation of serverbot.conf to accomodate new features in minor releases better #96. - Ignore dots in variables for feature eol #97. - Fixed bug that replaced a variable in serverbot.conf with only the first character of the source variable #99. - Changed variable when no serverbot.conf is available to be more clear. - Some minor text improvements across all files and comments. - Removed placeholders for planned features. - Made some preparations for method email. - Changed some default values in serverbot.conf to have less overlap.
sveeke
added a commit
that referenced
this issue
Sep 24, 2019
This is a combined update for all `1.1.0-UNSTABLE` commits. - Changed suffixes from `ALPHA`, `BETA` and `RELEASE` to new suffix layout `UNSTABLE` and `STABLE`. - Added feature EOL #63. - Added basic configuration file parameter validation #83. - Added notification to user on serverbot upgrade #84. - Removed abundance of comments that weren't very useful #85. - Added error when method telegram is not configured #88. - Made all gather functions calls consistent by placing them inside relevant functions #89. - Added branch variable in serverbot #90. - Removed argument shortcuts because of conflicting names #92. - Removed useless breaks in feature updates method cli #93. - Improved validation of serverbot.conf to accomodate new features in minor releases better #96. - Ignore dots in variables for feature eol #97. - Fixed bug that replaced a variable in serverbot.conf with only the first character of the source variable #99. - Changed variable when no serverbot.conf is available to be more clear. - Some minor text improvements across all files and comments. - Removed placeholders for planned features. - Made some preparations for method email. - Changed some default values in serverbot.conf to have less overlap.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I might have been a bit to enthusiastic with the amount of comments. Especially the comments for every if statement in a row of if statements can be removed.
The text was updated successfully, but these errors were encountered: