Hi Joel, Thanks for your review! Your concerns have been addressed, please verify that they are cleared.
Details inlined below. On Thu, Feb 19, 2026 at 2:06 AM Joel Halpern via Datatracker <[email protected]> wrote: > > Document: draft-ietf-netmod-yang-module-filename > Title: YANG module file name convention > Reviewer: Joel Halpern > Review result: Ready with Nits > > I am the assigned Gen-ART reviewer for this draft. The General Area > Review Team (Gen-ART) reviews all IETF documents being processed > by the IESG for the IETF Chair. Please treat these comments just > like any other last call comments. > > For more information, please see the FAQ at > > <https://wiki.ietf.org/en/group/gen/GenArtFAQ>. > > Document: draft-ietf-netmod-yang-module-filename-06 > Reviewer: Joel Halpern > Review Date: 2026-02-18 > IETF LC End Date: 2026-03-03 > IESG Telechat date: Not scheduled for a telechat > > Summary: This document is ready for publication as a Proposed Standard. > > Major issues: N/A > > Minor issues: N/A > > Nits/editorial comments: > In section 2, the text says that if one chooses to use the semver in the > file name, one uses it instead of the revision date. However, the syntax > diagram seems to say that both are permitted, with the revision date > coming > first. The intention is that the semver and revision dates are mutually exclusive in the file name. The draft has been updated to reflect this. See the SEC Dir review response for details. [0] > I wonder if there should be a comment about the risk of long semver > names causing file names to exceed some system limits? This has been omitted. It is assessed that there is no larger risk for long file names introduced than what other references already allow. E.g. the current file name form defined in RFC 7950 allows the yang-module-or-submodule part to be arbitrarily long, without any such consideration being presented. [0] https://mailarchive.ietf.org/arch/msg/netmod/mJ9jnRSf62Une-VOFLPgPhLoJBE/ -- Per _______________________________________________ netmod mailing list -- [email protected] To unsubscribe send an email to [email protected]
