Incorrect capitalization on this error, should be CLIError

NRLMMD-GEOIPS / geoips

Main Geolocated Information Processing System code base with basic functionality enabled.

https://nrlmmd-geoips.github.io/geoips/

Other

13 stars 10 forks source link

Incorrect capitalization on this error, should be CLIError #634

Open biosafetylvl5 opened 2 weeks ago

biosafetylvl5 commented 2 weeks ago

https://github.com/NRLMMD-GEOIPS/geoips/blob/6241dfb41e95f5d30888bcab7ca68e570cc45ce1/geoips/errors.py#L34

jsolbrig commented 1 week ago

This issue is technically correct, but I think it will require changes in a lot of the GeoIPS code. I think that we've frequently capitalized only the first letter for acronyms in class names. We should discuss this and document it in our standards.