-
-
Notifications
You must be signed in to change notification settings - Fork 33.5k
doc: added TC meeting minutes 2014-12-17 #178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Two outstanding items that need to be taken care of:
I'm noting them here because I don't have time to deal with them just yet! |
|
roadmap++ |
|
LGTM with the caveat that I wasn't there and haven't watched the video yet. :-) |
Closes #163 PR-URL: #178 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Bert Belder <[email protected]>
|
@rvagg I landed this PR - why reopen? |
|
@piscisaureus there's 2 items in the comments here that I still need to take care of, TODOs for me |
|
Ok got it |
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.
Blog of FileReader
Blob or FileReader?
The snprintf API is not available with older versions of MSVC. Found by a CI run.
Co-authored-by: François Doray <[email protected]>
Co-authored-by: François Doray <[email protected]>
Co-authored-by: François Doray <[email protected]>
Closes #163