adfinis / timedctl

GNU Affero General Public License v3.0
5 stars 0 forks source link

chore: Refactor timedctl into class #45

Closed gianklug closed 1 year ago

gianklug commented 1 year ago

I don't know whether this is worse or better than before. At least it would allow us to dynamically load config files or set a global debug flag in the future.

@C0rydoras what's your opinion on this?

I'm using classes mainly because this allows me to have a config and libtimed object in the scope of all functions that I can control from within a method without overwriting a global variable which is also quite ugly.

github-actions[bot] commented 1 year ago

Alex Recommends Report

Alex recommends the following language changes, but Alex is a regular expression based algorithm, so take them with a grain of salt.

:sparkles: :rocket: :sparkles: Nothing to Report :sparkles: :rocket: :sparkles: