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

suggest merge 'nbc-change-at','bc-change-at','editoral-change-at' into one extension #198

Open
lllyfeng opened this issue Feb 13, 2023 · 0 comments
Labels
schema-comparison Issues related to the YANG schema comparison draft

Comments

@lllyfeng
Copy link

'nbc-change-at','bc-change-at','editoral-change-at' are all used to describe the change information, it's difficult for implementing 3 extensions, because the change history should be list in time order regardless the change type, but every extension should be treated a indepent statement, YANG parser don't know the relationship between these 3 extensions.

I suggest use a one extension to express this information.

extension change-at {
argument revision-date-or-label;
}

and another extension 'change-type' can be used in extension 'change-at' as the sub-statement to indicate the compatible information.
extension change-type {
argument type;
description "the change-type can be nbc,bc,editorial".
}

for example:
list sir-changed-a-lot {
rev:change-at "3.0.0" {
rev:change-type "editorial";
}
rev:change-at "2.3.0" {
rev:change-type "bc";
}
rev:change-at "1.2.1_non_compatible" {
rev:change-type "bc";
}
description "a list of stuff";
ordered-by user;
key "foo";
leaf foo {
type string;
}
leaf thing {
type uint8;
}
}

@avtobiff avtobiff added the schema-comparison Issues related to the YANG schema comparison draft label Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
schema-comparison Issues related to the YANG schema comparison draft
Projects
None yet
Development

No branches or pull requests

2 participants