[Gen-art] Genart last call review of draft-eggert-bcp45bis-06

2021-10-31 Thread Elwyn Davies via Datatracker
Reviewer: Elwyn Davies
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

.

Document: draft-eggert-bcp45bis-06
Reviewer: Elwyn Davies
Review Date: 2021-10-31
IETF LC End Date: 2021-11-23
IESG Telechat date: Not scheduled for a telechat

Summary: Ready with nits.  The body of the document appears fine, but the
Abstract doesn't provide much insight into the content of the document.

Major issues:
None

Minor issues:
None

Nits/editorial comments:
Abstract:, para 1:  Suggest adding a sentence or two to explain what the
document actually does., e.g.:

s/... mailing list./...mailing list; this document provides guidance on the
intended scope of discussions appropriate for the mailing list as currently
envisaged while indicating certain types of postings that would be
inappropriate. In neither case are the lists considered exhaustive.   The
mailing list is intended to operate with self-moderation overseen by member(s)
of the Sergeants-at-Arms (SAA) team.  This document sets out the processes by
which SAAs monitoring this mailing list are appointed and removed together with
their powers as regards control of posting to the list../



___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art


[Gen-art] Genart last call review of draft-ietf-calext-ical-relations-08

2021-10-31 Thread Christer Holmberg via Datatracker
Reviewer: Christer Holmberg
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

.

Document: draft-ietf-calext-ical-relations-08
Reviewer: Christer Holmberg
Review Date: 2021-10-31
IETF LC End Date: 2021-10-28
IESG Telechat date: Not scheduled for a telechat

Summary: I have no technical issues with the draft, and the text is quite easy
to understand also for someone not familiar with the topic. However, I do have
some editorial comments (specific and general) that I would like the authors to
address.

Major issues: N/A

Minor issues: N/A

Nits/editorial comments:

Q1:

Please make sure that there are references on first occurrence.

For example, the first sentence of Section 1:

"iCalendar entities often..."

...should have a reference to RFC 5545.

In addition, I don't think we use references in the Abstract

---

Q2:

The Abstract says:

"This specification updates RELATED-TO defined in..."

I think it would be good to add a few words on HOW RELATED-TO is updated.

Also, please say "RELATED-TO property".

---

Q3:

Related to Q2, in Section 9 you say that you redefining RELATED-TO. I think
redefine and update are two separate things, so please clarify.

And, while Section 1.1 does explain how RELATED-TO is updated/redefined, it
would probably be good to have some text in Section 9 too. For example, if I
understand correctly you are updating Section 3.8.4.5. of RFC 5545. That should
be mentioned.

I would also consider changing the name of Section 9 to "Updates to RFC 5545",
and then in the text describe what those updates are.

---

Q4:

The text in Section 1.1 says:

"The currently existing iCalendar [RFC5545] RELATED-TO property has no support
for..."

Please avoid "currently", because it will have a different meaning depending on
when someone reads the spec.

Instead, I suggest to simply say:

"The iCalendar [RFC5545] RELATED-TO property has no support for..."



___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art