Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Fill implementation #61 #130

Merged
merged 2 commits into from
Oct 26, 2021

Conversation

OlivierCavadenti
Copy link
Contributor

@holgerbrandl
Copy link
Owner

2 questions

  • Since this is not part of the core-interface, this could also live in an extension function (similar to the rest of the complementary API), couldn't it?
  • Shouldn't we better use vararg columSelects: ColumnSelector instead of vararg columnsToFill: String? Both for consistency with select and most other selecting methods, but also because of its enhanced functionality.

@OlivierCavadenti
Copy link
Contributor Author

2 questions

  • Since this is not part of the core-interface, this could also live in an extension function (similar to the rest of the complementary API), couldn't it?

Yes, I move it to Extensions.kt

  • Shouldn't we better use vararg columSelects: ColumnSelector instead of vararg columnsToFill: String? Both for consistency with select and most other selecting methods, but also because of its enhanced functionality.

I make 2 functions with ColumnSelector and List to have both options.

@holgerbrandl holgerbrandl merged commit d30a370 into holgerbrandl:master Oct 26, 2021
@holgerbrandl
Copy link
Owner

Awesome, thanks for this wonderful contribution.

@OlivierCavadenti
Copy link
Contributor Author

Awesome, thanks for this wonderful contribution.

No problem, I will check lately for other functionnalities to implements.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

provide a tidyr::fill equivalent
2 participants