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

Fixed defaults function output #80

Closed
wants to merge 1 commit into from

Conversation

steweg
Copy link
Contributor

@steweg steweg commented Jan 25, 2024

This patch fixes output for leaf/leaf-list nodes
using decimal64 type. It also fixes the output of
leaf-list defaults and add new unit test for this
purpose

This patch fixes output for leaf/leaf-list nodes
using decimal64 type. It also fixes the output of
leaf-list defaults and add new unit test for this
purpose
@steweg steweg mentioned this pull request Jan 25, 2024
samuel-gauthier pushed a commit that referenced this pull request Jan 25, 2024
This patch changes the output for leaf/leaf-list nodes using decimal64
type, and adds new unit test for this purpose.

The leaf-list defaults function is reworked to be similar to the leaf
default function. A new yang schema is added for clarity, and to avoid
modifying all the tests using yolo-system.yang.

Fixes: #80
Signed-off-by: Stefan Gula <[email protected]>
Signed-off-by: Samuel Gauthier <[email protected]>
@samuel-gauthier
Copy link
Collaborator

samuel-gauthier commented Jan 25, 2024

I split your commit into 3428ea0 and 3b4d509, and reworded for clarity. Thanks!

@steweg steweg deleted the bugfix/defaults_fix branch January 31, 2024 07:47
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