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

Remove revision-label and revision-label-scheme from module versioning draft #210

Closed
rgwilton opened this issue Nov 15, 2023 · 0 comments
Closed

Remove revision-label and revision-label-scheme from module versioning draft #210

rgwilton opened this issue Nov 15, 2023 · 0 comments
Assignees
@rgwilton
Labels
Design change needs a change to the actual design / function updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft)

Comments

@rgwilton
Copy link
Collaborator

T3. revision-label-scheme extension (Sec 3.4.2)
Removed. Based on WG LC discussions we will go back to a single versioning scheme for YANG modules, and hence the revision-label-scheme extension will be removed from this draft.

T4. revision-label extension (Sec 3.4)
Removed. Related to T3 above, given that a single versioning scheme is sufficient, the revision-label extension will be moved to the YANG Semver draft, and removed from Module Versioning.
@rgwilton rgwilton self-assigned this Nov 15, 2023
@rgwilton rgwilton added Design change needs a change to the actual design / function updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft) labels Nov 15, 2023
rgwilton added a commit that referenced this issue Nov 15, 2023
@rgwilton
Removes revision-label & scheme, fixes #210
29d603f
Reshad-Rahman added a commit that referenced this issue Mar 1, 2024
@rgwilton @Reshad-Rahman
Ietf 118 mod ver updates (#217)
454b3fe 
* Fix #207

* Remove contentious whitespace text #208

* Fix T5, ambiguous imports #209

* Removes revision-label & scheme, fixes #210

* Reformat source file + regenerate

* Soem changes to support #211

* Proposed text for #221

* Renamed ietf-yang-library-revisions to ietf-yang-status-conformance
#215

* Fixed YANG validation issue caused by extra " in description of recommended-min-date
#225

While in there, also replaced remaining instances of recommended-min by recommended-min-date.
Removed rev:label from the 2 YANG modules since the label extension is gone.

* Removed remaining instances of revision-label-scheme.
#224

* For "recommended-min-date", do a date comparison. Modified other related text.

#213

* Making yang-semver an informative reference in module-versioning

#212

* recommended-min-date can not be used multiple times per import statement.

#228

* Reorder labels as per #211

* Minor tweaks during Feb 20 call

* Removed text which mentions labels (except for the example in 3.4)
#227

Some fixes wrt "recommended-min-date"
#213

Small cleanups too.

* Fixed incorrect yl-rev prefix.

* Further minor changes and cleanup

* Change elided to ommitted, update .txt file

* Addressed comments from Rob and Jason.

* Incorrectly put " in YANG description, replaced with '

---------

Co-authored-by: Reshad Rahman <reshad@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rgwilton rgwilton

Labels
Design change needs a change to the actual design / function updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rgwilton