Fix Version/s: None
Team:Telescope and Site
There are many camelCase variables and method arguments that need to be renamed to snake_case. This ticket is for doing that.
This lists camelCase occurences: grep -r '\<[a-z]\w*[A-Z]\w*\>' *
Even LSST style guide allows for deviations if it matches the existing API - for that reason I keep camelCase in my Qt based code, as Qt is camelCase (from C++ API down to Python). I made a few changes, but keep most of the camelCase as those names originate from various camera API. It doesn't make any sense to change this mapping.
I know the topic title and description only refer to variables but the idea is to also make sure that the method names are snake_case. The tricky part is to leave those that refer to native library variables and methods in place.
Well we can decorate API methods with e.g. @api decorator (which will not be defined) to distinguish which doesn't need to be snake_case?
Reviewed on GitHub. I left a few comments to improve this even more but otherwise you did a very nice job!
This was partially done by me in
DM-33927but needs to be revised to see what's remaining.