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

Identity classes in YDK-Py and YDK-Cpp do not use CapWords #538

Closed
111pontes opened this issue Sep 6, 2017 · 1 comment
Closed

Identity classes in YDK-Py and YDK-Cpp do not use CapWords #538

111pontes opened this issue Sep 6, 2017 · 1 comment
Assignees
@ylil93
Labels
bug

Comments

@111pontes
Copy link
Collaborator

111pontes commented Sep 6, 2017 •
edited
Loading

The naming of identity classes doesn follow CapWords in YDK-Py 0.6.0. For instance:
http://ydk.cisco.com/py/docs/gen_doc_8afdd1da8f494fd1f80e8849f77bfb6034464cee.html
http://ydk.cisco.com/cpp/docs/gen_doc_8afdd1da8f494fd1f80e8849f77bfb6034464cee.html
@111pontes 111pontes added the bug label Sep 6, 2017
@111pontes 111pontes changed the title Identity classes in YDK-Py do not use CapWords Identity classes in YDK-Py and YDK-Cpp do not use CapWords Sep 13, 2017
@ghost ghost assigned ylil93 Sep 14, 2017
@ylil93
Copy link
Contributor

ylil93 commented Sep 14, 2017 •
edited
Loading

The current openconfig bundle that is publicly available via pip is outputting "AddPathsIdentity" for that class. However the documentation generated from source code on the master branch is outputting "ADD_PATHS". I can correct output be "AddPaths"

Edit:
Seems like the name in the yang model itself has changed such that the name is capitalized, so I changed the output to be "ADDPATHS"

ylil93 added a commit to ylil93/ydk-gen that referenced this issue Sep 15, 2017
@ylil93
update class naming mechanics (CiscoDevNet#538)
62026aa
@ylil93 ylil93 closed this as completed Sep 15, 2017
ghost pushed a commit that referenced this issue Sep 18, 2017
Merge pull request #553 from ylil93/master
5c7608f 
Fixed Github issues #538 #528
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ylil93 ylil93

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ylil93 @111pontes