Skip to content

Updated usages of dataFrameOf(vararg columns: AnyBaseCol) in samples#1770

Merged
koperagen merged 1 commit intomasterfrom
dataframeof-migration
Mar 26, 2026
Merged

Updated usages of dataFrameOf(vararg columns: AnyBaseCol) in samples#1770
koperagen merged 1 commit intomasterfrom
dataframeof-migration

Conversation

@koperagen
Copy link
Copy Markdown
Collaborator

fixes #1146

@koperagen koperagen added this to the 1.0.0-Beta5 milestone Mar 26, 2026
@koperagen koperagen requested a review from Jolanrensen March 26, 2026 15:24
@koperagen koperagen self-assigned this Mar 26, 2026
@koperagen koperagen added the documentation Improvements or additions to documentation (not KDocs) label Mar 26, 2026
df.explode { a and b }
val df = dataFrameOf(
"a" to columnOf(listOf(1, 2), listOf(3, 4, 5)),
"b" to columnOf(listOf(1, 2, 3), listOf(4, 5))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

trailing comma, ktlint may start complaining :)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same below

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

actually... why doesn't it complain, huh

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

i.. don't know :) i even ran linter before. checked again - seems to ignore this

Copy link
Copy Markdown
Collaborator Author

@koperagen koperagen Mar 26, 2026

Choose a reason for hiding this comment

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

@Suppress("ktlint:standard:chain-method-continuation", "ktlint:standard:argument-list-wrapping") in this file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

weird :)

@koperagen koperagen force-pushed the dataframeof-migration branch from 1d9b4b2 to e167032 Compare March 26, 2026 16:09
@koperagen koperagen merged commit 1fcbabc into master Mar 26, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation (not KDocs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate examples to use APIs supported in compiler plugin

2 participants