w3c / scribe2

Converts IRC minutes to HTML
Other
11 stars 7 forks source link

Please continue to include failed s/// commands #23

Closed swickr closed 2 years ago

swickr commented 2 years ago

I notice a TODO in the source:

8 # TODO: Omit` failed s/// commands?

I find the in-line inclusion of failed substitutions helpful when I am cleaning the minutes. I read the diagnostics for any failed substitutions and then search for them in-line to locate the point in the minutes where the proposed substitution was made to determine when some form of correction might still be appropriate.

So please do not omit them.

bert-github commented 2 years ago

I removed the TODO.