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

add readline(::AbstractCmd) #34927

Merged
merged 3 commits into from
Mar 1, 2020
Merged

Conversation

bicycle1885
Copy link
Member

This adds the readline(::AbstractCmd) method in a trivial way, unless I miss something non-trivial. This closes #28975.

@kshyatt kshyatt added the io Involving the I/O subsystem: libuv, read, write, etc. label Feb 29, 2020
@JeffBezanson JeffBezanson merged commit 570f6d9 into JuliaLang:master Mar 1, 2020
@Keno
Copy link
Member

Keno commented Mar 1, 2020

This broke win64 CI. I'm gonna revert this quickly to get the buildbot back to green, but please resubmit this and we can work on getting this to work without breaking the tests.

Keno added a commit that referenced this pull request Mar 1, 2020
Keno added a commit that referenced this pull request Mar 2, 2020
ravibitsgoa pushed a commit to ravibitsgoa/julia that referenced this pull request Apr 9, 2020
ravibitsgoa pushed a commit to ravibitsgoa/julia that referenced this pull request Apr 9, 2020
KristofferC pushed a commit that referenced this pull request Apr 11, 2020
KristofferC pushed a commit that referenced this pull request Apr 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
io Involving the I/O subsystem: libuv, read, write, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

readline not working for ls, but readlines work
4 participants