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

feat: insertcols! receives kwarg after #2829

Merged
merged 7 commits into from Sep 1, 2021

Conversation

pstorozenko
Copy link
Contributor

Closes #2613

@bkamins bkamins added feature non-breaking The proposed change is not breaking labels Aug 2, 2021
@bkamins bkamins added this to the 1.3 milestone Aug 2, 2021
NEWS.md Show resolved Hide resolved
Co-authored-by: Bogumił Kamiński <bkamins@sgh.waw.pl>
src/dataframe/dataframe.jl Outdated Show resolved Hide resolved
src/dataframe/dataframe.jl Outdated Show resolved Hide resolved
test/dataframe.jl Outdated Show resolved Hide resolved
Copy link
Member

@bkamins bkamins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Let us wait a few days for other people to comment.

pstorozenko and others added 2 commits August 2, 2021 17:13
Co-authored-by: Bogumił Kamiński <bkamins@sgh.waw.pl>
@bkamins
Copy link
Member

bkamins commented Aug 25, 2021

@pstorozenko - this requires a simple merge conflict fix.

@nalimilan - could you please review the PR? Thank you!

src/dataframe/dataframe.jl Outdated Show resolved Hide resolved
Co-authored-by: Milan Bouchet-Valat <nalimilan@club.fr>
@bkamins
Copy link
Member

bkamins commented Aug 29, 2021

Thank you. I will merge it once the CI passes.

@bkamins bkamins merged commit f9ca4ad into JuliaData:main Sep 1, 2021
@bkamins
Copy link
Member

bkamins commented Sep 1, 2021

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature non-breaking The proposed change is not breaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add after keyword argument to insertcols!
3 participants