-
Notifications
You must be signed in to change notification settings - Fork 111
Conversation
@joyent/node-coreteam FYI |
The github link for me should be https://github.com/mhdawson |
|
||
MD: this is a tempoary fix, most people (hopefully) are using defaults. | ||
|
||
TJ: this is more in minor versions, when peopel are bringing up the app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
peopel
-> people
. Although I'm not sure what the whole sentence refers to.
Review notes incorporated |
|
||
TJ: this should be first class documentation. | ||
|
||
- [ ] srl, open a new ticket. Document that Date.now != how timers work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we could replace [ ]
with: Action item:...
?
|
||
#### Wrong checksum for node v0.12.2 | ||
|
||
* This issue will be investigated. (by whom?) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for bullet point, issue was investigated by me.
@srl295 Thank you! Please squash all the commits in one commit that has "Add minutes from 2015-04-16 meeting" as a commit message. Once that is done, LGTM! |
@misterdjules thank you! |
LGTM, thank you @srl295. |
Landed in 90ad03c, thanks again! |
Minutes..