search

DeepTrackAI / deeplay

Other
5 stars 6 forks source link

Lowercase Module Names #42

Closed giovannivolpe closed 11 months ago

giovannivolpe commented 11 months ago

According to PEP 8, module names should be lowercase. Shall we change this consistently in deeplay?

cmanzo commented 11 months ago

I think we should

BenjaminMidtvedt commented 11 months ago

Which modules are not already lower case?

Edit: ok the block/PLAN etc? I'm ok with changing those to lower case.

cmanzo commented 11 months ago

Blocks are now lowercase in PR #40

cmanzo commented 11 months ago

Fixed in PR #40