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

Update base to current pyang master (2.6.1 plus something) #9

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

cabo
Copy link
Member

@cabo cabo commented Feb 22, 2025

No description provided.

Nick Hancock and others added 30 commits October 30, 2023 09:25
Add <<enumeration>> stereotype to an enumeration class
Add support for rendering a typedef of bits as a bits class
Add support for defining the max number of bit values to render in the bits class.
Add support to indicate that more enum or bit values are not shown by means of an ellipsis
This refactoring allows for more alternatives in the future.
Remove unnecessary variables. Add support for association.
Shorten and simplify descriptions of new UML options where possible.
Align more with existing descriptions, even if not 100% correct.
Simplify code where applicable.
…ml-no

Also correct error in uml-case which references the wrong set of strings, if the option includes an invalid string.
Exitsing ctx_title renamed ctx_title_text so that ctx_title could be used for uml-no.
Correct default assignment to glob in function emit_grouping, should be Boolean not string.
Modifies the info message to quote the name of the grouping rather than the PlantUML keyword of the parent node of the uses statement, which may not be understandable to uses not familiar with PlantUML code.
… keywords

PlantUML does not accept dots in keywords, but the current UML plugin does not convert dots in keywords (as permitted for YANG identifiers by RFC 7950 section 6.2) to underscores.
Also to ensure there is no chance of a clash of keywords where say both the identifiers 'a.b' and 'a-b' exist, use 2 underscores for dots.
Whether further child statements should be rendered depends on whether cont=true when emit_child_stmt() is called and by augments it is set to false, but emit_notify() always emits child statements. So this change moves the the call to emit_child_stmt() out of emit_notify() in the same way that this is done for lists.
* Replace sys.exit with return, and create an error object for the IOError prior to returning.

If other modules import this code, an exit in a library may have
unintended consequences.  Like in other places, use return instead of
exit.

---------

Co-authored-by: Joe Clarke <[email protected]>
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.

6 participants