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

End deprecation period of eachcol #1752

Merged
merged 8 commits into from
Apr 1, 2019
Merged

Conversation

bkamins
Copy link
Member

@bkamins bkamins commented Mar 26, 2019

I had to open a new PR for this (the old one was #1613).
It is now rebased to the current master.

@bkamins bkamins requested a review from nalimilan March 26, 2019 10:19
src/abstractdataframe/iteration.jl Show resolved Hide resolved
src/abstractdataframe/abstractdataframe.jl Outdated Show resolved Hide resolved
src/abstractdataframe/iteration.jl Outdated Show resolved Hide resolved
@bkamins bkamins mentioned this pull request Mar 30, 2019
@bkamins
Copy link
Member Author

bkamins commented Mar 30, 2019

@nalimilan I have dropped , false in eachcol everywhere expect where it was tested to work correctly. This should be good to merge now. Then I can rebase "ownership" PR with this change.

docs/src/lib/types.md Outdated Show resolved Hide resolved
src/abstractdataframe/abstractdataframe.jl Outdated Show resolved Hide resolved
src/abstractdataframe/iteration.jl Outdated Show resolved Hide resolved
test/indexing.jl Outdated Show resolved Hide resolved
test/indexing.jl Outdated Show resolved Hide resolved
@bkamins
Copy link
Member Author

bkamins commented Apr 1, 2019

Thank you for finding more, 😄 - this was exactly my problem. I have grep-ed them, but there were so many places to fix that I must have missed something (as , false could appear also in other contexts)

@bkamins bkamins merged commit d4dfbb4 into JuliaData:master Apr 1, 2019
@bkamins bkamins deleted the sync_eachcol branch April 1, 2019 18:01
@bkamins bkamins mentioned this pull request May 7, 2019
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants