Skip to content
This repository was archived by the owner on Aug 1, 2019. It is now read-only.

Created a proposal roadmap for the thread summary process - #243

Merged
bnb merged 2 commits into
nodejs:masterfrom
vdeturckheim:master
Nov 30, 2016
Merged

bnb merged 2 commits into
nodejs:masterfrom
vdeturckheim:master

Conversation

@vdeturckheim

Copy link
Copy Markdown
Member

This follows discussions in #241 regarding the writing of articles summarizing of long threads within the organisation.

@bnb

bnb commented Oct 23, 2016

Copy link
Copy Markdown
Contributor

Pretty comprehensive list - LGTM.

@trevnorris, @jasnell, and @nodejs/ctc: Do you two have any further feedback on what you would like to see from this?

@vdeturckheim

Copy link
Copy Markdown
Member Author

@trevnorris @jasnell , would you have time for a quick review this week? :)

@jasnell

jasnell commented Nov 6, 2016

Copy link
Copy Markdown
Member

Absolutely

On Sunday, November 6, 2016, Vladimir de Turckheim notifications@github.com
wrote:

@trevnorris https://github.com/trevnorris @jasnell
https://github.com/jasnell , would you have time for a quick review
this week? :)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#243 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa2eWLwFh9HiuHAgQRa9ToQFS1zkhFwks5q7eYkgaJpZM4KeKhX
.

@Fishrock123

Copy link
Copy Markdown

Not particularly clear, but seems like a starting point!

@trevnorris

Copy link
Copy Markdown

Cool. I was thinking something as simple as after 50 comments a summary is written for the previous 50. Also give them a special tag so my script can collapse everything down that's not a summary.

Overall this looks good.

@bnb

bnb commented Nov 30, 2016

Copy link
Copy Markdown
Contributor

Looks like we have general consensus. Merged! ❤️

@bnb
bnb merged commit 2166f2a into nodejs:master Nov 30, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants