You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
These would for example be created for paths: /v2/abc and /v2/abc/deFgHij
The results however inserts an additional underscore between v and 2, so v_2 instead of v2.
Expected behavior
results should be: get_v2_abc_de_fg_hij
Desktop (please complete the following information):
OS: CentOS7
Python Version: 3.8
openapi-python-client version: github-master from today
Suggestion: snake_case uses split_words function , which only splits on Aa, so a capital letter followed by a lowercase one, not on underscore. split_words should first split on underscore, then split on capital followed by lowercase. This would solve the issue of mixed snake and camelcase names.
These of course violate camelCase, but in my estimate should all be treated as one word, i.e. multiple consecutive capital letters are treated as a single block. So above model4dd would be the outcome in both cases.
The text was updated successfully, but these errors were encountered:
Describe the bug
The names of apis are odd, having '_' in irregular and unpredictable places
To Reproduce
The following behavior can be observed using the
snake_case
function from utils:These would for example be created for paths:
/v2/abc
and/v2/abc/deFgHij
The results however inserts an additional underscore between
v
and2
, sov_2
instead ofv2
.Expected behavior
results should be:
get_v2_abc_de_fg_hij
Desktop (please complete the following information):
Suggestion:
snake_case
usessplit_words
function , which only splits onAa
, so a capital letter followed by a lowercase one, not on underscore.split_words
should first split on underscore, then split on capital followed by lowercase. This would solve the issue of mixed snake and camelcase names.Some additional odd outcomes:
These of course violate camelCase, but in my estimate should all be treated as one word, i.e. multiple consecutive capital letters are treated as a single block. So above
model4dd
would be the outcome in both cases.The text was updated successfully, but these errors were encountered: