Skip to content

Commit d7c0f96

Browse files
committed
Process Resolved: Open for things that need WG review; improve documentation
1 parent 75769ed commit d7c0f96

1 file changed

Lines changed: 83 additions & 5 deletions

File tree

bin/issuegen.pl

Lines changed: 83 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,27 +17,105 @@
1717

1818
my $inFile = $ARGV[0];
1919
if (!$inFile) {
20-
print "\nPass in issues list filename for processing!\n\n";
20+
print "\nPass in issues list filename for processing!";
21+
print "\nOr use argument 'help' for help.\n\n";
2122
print "~~~~~~~~~~~~~~~~~~~~~ Template for issues-list.txt ~~~~~~~~~~~~~~~~~~~~~\n";
2223
print <<XXX;
2324
2425
Draft: http://www.w3.org/TR/2013/WD-css-text-decor-3-20130103/
2526
Title: CSS Text Decoration Level 3
26-
... anything else you want here, except 4 dashes ...
27+
... any notes you want here, except 4 dashes ...
2728
2829
----
2930
Issue 1.
3031
Summary: [summary]
3132
From: [name]
3233
Comment: [url]
3334
Response: [url]
34-
Closed: [Accepted/OutOfScope/Invalid/Rejected/Retracted ... or replace this "Closed" line with "Open"]
35+
Closed: [Accepted/OutOfScope/Invalid/Rejected/Retracted/Deferred ... or replace this "Closed" line with "Open"]
3536
Verified: [url]
3637
Resolved: Editorial/Bugfix (for obvious fixes)/Editors' discretion/[url to minutes]
3738
----
3839
XXX
3940
exit;
4041
}
42+
if ($inFile eq 'help') {
43+
print <<XXX;
44+
45+
Welcome to fantasai's Issues List Generator!
46+
47+
This is a script that converts a plaintext (.txt) issues list into a
48+
color-coded HTML file of the same name (but .html file extension).
49+
The input is itself a presentable, easily-editable file designed
50+
mostly for the editor’s convenenience.
51+
52+
The original purpose of this format is to create a Disposition of
53+
Comments for the W3C's LCWD->CR transition process. However, it is
54+
also useful for tracking issues in general and can be used as such.
55+
There is no requirement to use this format; fantasai merely found it
56+
the most convenient way to track issues and create a DoC.
57+
58+
Beyond the header, the script itself only processes the dividers,
59+
the issue number, and the status (Closed/Open). Additional fields
60+
may be added or removed as desired; they are merely passed through.
61+
62+
fantasai suggests the following fields, as seen in the template:
63+
64+
Summary: A one-line summary of the issue, so that a reviewer can
65+
quickly grasp what the issue was about.
66+
67+
From: The name of the commenter.
68+
69+
Comment: URL to the message that most clearly presents the issue.
70+
Usually the initial message in a thread, but not always.
71+
This is the hook into the discussion, where further details
72+
can be read.
73+
74+
Response: URL to the editor’s response that is intended to close
75+
the issue: usually either reporting changes made to the
76+
spec, or explaining why changes aren't being made.
77+
Note this is not always the first reply from the editor.
78+
79+
The Comment/Response lines can be repeated if new responses are
80+
made presenting information that reopens the issue; however the
81+
entire thread shouldn't be tracked, only key messages. The goal
82+
is to minimize the effort required for someone reviewing the issues
83+
to understand this issue and its resolution.
84+
85+
Closed: A status line on how the issue was closed. Triggers colors.
86+
87+
Verified: URL to a message where the commenter indicates satisfaction
88+
with the resolution of the issue.
89+
(If the status was Rejected, this will turn the color green.)
90+
It's helpful to get the commenter's feedback on the changes,
91+
since they will often notice any mistakes. Verification
92+
indicates full closure of the issue.
93+
94+
Resolved: I use this line to track by what authority the issue was closed.
95+
It makes me think explicitly about whether the WG or anyone
96+
else should be consulted for solutions/review/approval.
97+
Values applicable to the CSSWG include:
98+
Editorial - no substantive change
99+
Bugfix - fixes an obvious error with an obvious solution
100+
[URL] - link to WG resolution closing the issue
101+
Editor's discretion -
102+
This is the tricky one. It's used in cases where
103+
1. the solution isn't obvious (not Bugfix)
104+
2. the impact of the solution is minor and localized:
105+
* there is no cross-module impact
106+
* there is no cross-module consistency concern
107+
* it is unlikely to affect implementation architecture
108+
3. no syntax/API is affected
109+
4. there is consensus on the mailing list, at least
110+
among the people involved in the discussion; and
111+
nobody not involved is likely to care
112+
It is also occasionally used to close issues as No Change
113+
in cases where the commenter is clueless or the requested
114+
change would clearly violate a WG design principle.
115+
~fantasai
116+
XXX
117+
exit;
118+
}
41119

42120
# Input/Output setup
43121
my $outFile = $inFile;
@@ -73,7 +151,7 @@
73151
elsif (/\n(?:Closed|Open):\s+(\S+)/) {
74152
$code = $statusStyle{lc $1};
75153
}
76-
if (/\nOpen/) {
154+
if (/\nOpen/ or /\nResolved:\s+Open/) {
77155
$code .= ' ' if $code;
78156
$code .= 'open';
79157
}
@@ -170,4 +248,4 @@ sub script {
170248
}());
171249
</script>
172250
XXX
173-
}
251+
}

0 commit comments

Comments
 (0)