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

IETF module rules - #221

Open
rgwilton opened this issue Dec 12, 2023 · 0 comments
Open

IETF module rules - #221

rgwilton opened this issue Dec 12, 2023 · 0 comments
Assignees
@rgwilton
Labels
updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft) yang-semver-soln Issues that apply to yang-semver (yang semver solution draft)

Comments

@rgwilton
Copy link
Collaborator

Reword 6.1, perhaps also move some text to Semver draft

6.1. Guidelines for YANG module authors

All IETF YANG modules MUST include revision label statements for all
newly published YANG modules, and all newly published revisions of
existing YANG modules. The revision label MUST take the form of a
YANG semantic version number [I-D.ietf-netmod-yang-semver].

This draft =

  • IETF modules MUST conform to the this draft (sections X, Y,Z).
@rgwilton rgwilton added yang-semver-soln Issues that apply to yang-semver (yang semver solution draft) updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft) labels Dec 12, 2023
rgwilton added a commit that referenced this issue Jan 16, 2024
@rgwilton
Proposed text for #221
d52a9e9
@rgwilton rgwilton self-assigned this Jan 16, 2024
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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rgwilton rgwilton

Labels
updated-mod-rev-handling Issues related to yang-module-versioning (updated module revision handling draft) yang-semver-soln Issues that apply to yang-semver (yang semver solution draft)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rgwilton