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

Snake case conversion enhancement #12

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Conversation

v0-e
Copy link
Contributor

@v0-e v0-e commented Apr 12, 2024

This should keep the same behavior of the current snake case conversion, however it now adds only one _ to a sequence of uppercase letters instead of one _ per uppercase letter, e.g.,
ASN.1 identifier: messageID, from message_i_d to message_id
ASN.1 module: ETSI-ITS-CDD, from e_t_s_i__i_t_s__c_d_d to etsi__its__cdd

Copy link
Member

@6d7a 6d7a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thank you for your contribution!

@6d7a 6d7a merged commit cf4ee90 into librasn:main Apr 12, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants