Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Python doesn't want CamelCase #45

Closed
OGKevin opened this issue Nov 15, 2017 · 0 comments
Closed

Python doesn't want CamelCase #45

OGKevin opened this issue Nov 15, 2017 · 0 comments
Assignees
@OGKevin
Milestone
0.12.4

Comments

@OGKevin
Copy link
Contributor

OGKevin commented Nov 15, 2017

Steps to reproduce:

  1. Look at https://github.com/bunq/sdk_python/blob/develop/tests/model/generated/object/test_notification_url.py#L94

What should happen:

  1. The words are separated with _

What happens:

  1. The methods are written in camelCase 🤦‍♂️

Logs

  • Logs 'no logs`

Extra info:
@OGKevin OGKevin added this to the 0.12.4 milestone Nov 15, 2017
@OGKevin OGKevin self-assigned this Nov 15, 2017
OGKevin added a commit that referenced this issue Nov 16, 2017
@OGKevin
Renamed camelCase methods. (#45)
27d454f
@OGKevin OGKevin mentioned this issue Nov 16, 2017
Merged
dnl-blkv added a commit that referenced this issue Nov 21, 2017
@dnl-blkv
Merge pull request #48 from bunq/feature/no_camelCase_in_python_#45
d288c92 
Renamed camelCase methods. (#45)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OGKevin OGKevin

Labels
None yet
Projects
None yet
Milestone
0.12.4
Development

No branches or pull requests
1 participant
@OGKevin