Local function names should match - Githubissues

slgobinath / SafeEyes

Protect your eyes from eye strain using this simple and beautiful, yet extensible break reminder

http://slgobinath.github.io/SafeEyes/

GNU General Public License v3.0

1.45k stars 163 forks source link

Local function names should match #550

Closed AdamPS closed 10 months ago

AdamPS commented 10 months ago

Is your feature request related to a problem? Please describe.

For example we have code like this: enable_safe_eyes = context['api']['enable_safeeyes']

The local name has an extra _ compared with the actual function name. This generates confusion for developers:

Searching for uses of enable_safeeyes( misses some cases
Looking for the function definition of enable_safe_eyes doesn't work.

Describe the solution you'd like Fix names to match