financial_life
financial_life copied to clipboard
MartinPyka
Use CapWords for class names
It's the Python standard in PEP-8, and having classes with Cap_Words (e.g., Bank_Account) is confusing. Since the API is not finalized yet, do you think it's possible to change those to BankAccount etc, and maybe keep the old ones, but with a deprecation warning?
yes, it is still possible and I would support it. For now, I have no time to change this, but I think I put this on my agenda.
I had your repo on my to-help list for months. Most of those issues I have an idea how to solve, and will be submitting PRs when I have time.
On Wed, Jul 5, 2017, 13:44 MartinPyka [email protected] wrote:
yes, it is still possible and I would support it. For now, I have no time to change this, but I think I put this on my agenda.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MartinPyka/financial_life/issues/8#issuecomment-313206335, or mute the thread https://github.com/notifications/unsubscribe-auth/AEvDPM53Hyp4wpaHJfa9hBSEM6LIokoOks5sK-eRgaJpZM4ON8iI .
--
勝者神優