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.

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

feature flag to allow allow changing of '.' to '_' in the normalize_symbol function #66

Open
ITJamie opened this issue Mar 19, 2024 · 0 comments

Comments

@ITJamie
Copy link
Contributor

ITJamie commented Mar 19, 2024

when using some swagger apis the operationid can have long paths (example ) eg "operationId": "cmk.gui.plugins.openapi.endpoints.notification_rules.show_rules",

this was creating service functions with slightly unreadable names:
eg:def cmkguipluginsopenapiendpointsnotification_rulesshow_rules

by patching openapi_python_generator/language_converters/python/common.py function normalize_symbol and adding symbol = symbol.replace(".", "_") we get a saner function name of:
def cmk_gui_plugins_openapi_endpoints_notification_rules_show_rules

while this change might not be what most people want. having a toggle for it would be quite useful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant