Skip to content
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

RFC: improve docs for IOContext. fixes #16763 #17545

Merged
merged 1 commit into from
Jul 22, 2016
Merged

Conversation

JeffBezanson
Copy link
Member

(also remove some references to the removed multiline property)

Suggestions welcome on what or where else to put in docs on this.

@tkelman
Copy link
Contributor

tkelman commented Jul 22, 2016

cc @dpsanders do you think this is enough information for package authors to be able to use this effectively?

@timholy
Copy link
Member

timholy commented Jul 22, 2016

Perhaps document the kwarg syntax too?

@StefanKarpinski StefanKarpinski added this to the 0.5.0 milestone Jul 22, 2016
@StefanKarpinski StefanKarpinski added docs This change adds or pertains to documentation io Involving the I/O subsystem: libuv, read, write, etc. labels Jul 22, 2016
also remove some references to the removed `multiline` property
@JeffBezanson
Copy link
Member Author

@timholy Added, and also mentioned displaysize property.

I'm cancelling CI since the non-doc part of the change already passed on a few platforms.

@StefanKarpinski
Copy link
Member

I would suggest that we merge this and close the issue for now – there's nothing preventing further documentation improvements in the 0.5.x timeframe. It's time to get on with the release.

@JeffBezanson JeffBezanson merged commit c849f54 into master Jul 22, 2016
@tkelman tkelman deleted the jb/iocontextdoc branch July 22, 2016 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation io Involving the I/O subsystem: libuv, read, write, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants