w3c / scribejs

Converter of RRSAgent IRC logs into minutes in markdown
https://w3c.github.io/scribejs/BrowserView/
Other
11 stars 14 forks source link

Better scribe handling using scribe "pools" #75

Closed iherman closed 5 years ago

iherman commented 5 years ago

@bigbluehat,

I have added a new scribing model to the script, inspired by a similar change that has been done by the core scribe script that is also used elsewhere at W3C. This PR should not be merged yet, because I would prefer to test it a bit more, although it seems to work now, and I have also used it on the latest JSON-LD WG call minutes (I have just modified the log a bit to turn it into a test).

The new scribing approach is as follows.

This change, while fully backward compatible with current practice. But it is very helpful when a temporary scribe replacement is needed, because the original scribe wants to say something:

<pchampin> scribenick: pchampin
...
<pchampin> bigbluehat: says something great
<rubenworks> scribe+
<rubenworks> pchampin: says something specific, not related to scribing, 
<rubenworks> ... and it is now part of the minutes
...
<pchampin> ivan: says something else (taking over minuting without further ado)
<pchampin> ... minutes are continued
<rubenworks> ... can be continued by the other scribe...

Note that it is no longer necessary to set <pchampin> scribenick: pchampin "back", and then again set it for Ruben, etc.

WDYT? Is this a worthwhile addition to the script or it would just mess up people? Is the functionality clear at least to you?

iherman commented 5 years ago

Modified the functionality a little bit in https://github.com/w3c/scribejs/pull/75/commits/abd5616fafcc084d27572fb340f506c8f66b0146: introduced 'command=' to wipe out old content, and 'command:' is equal to 'command+'. This is valid for all such terms, in particular to scribes.