Skip to content

Apply korro to samples in data sources#1833

Merged
Allex-Nik merged 4 commits into
masterfrom
data-sources-samples-korro
May 19, 2026
Merged

Apply korro to samples in data sources#1833
Allex-Nik merged 4 commits into
masterfrom
data-sources-samples-korro

Conversation

@Allex-Nik
Copy link
Copy Markdown
Collaborator

@Allex-Nik Allex-Nik self-assigned this Apr 29, 2026
@Allex-Nik Allex-Nik added the documentation Improvements or additions to documentation (not KDocs) label Apr 29, 2026
@Allex-Nik
Copy link
Copy Markdown
Collaborator Author

Is the package correct for the samples? Should ReadSQLDatabases be in api instead of io?

Comment thread docs/StardustDocs/topics/readSqlDatabases.md
Comment thread docs/StardustDocs/topics/dataSources/Parquet.md
Comment thread docs/StardustDocs/topics/dataSources/Parquet.md Outdated
Comment thread samples/src/test/kotlin/org/jetbrains/kotlinx/dataframe/samples/api/Read.kt Outdated
Comment thread docs/StardustDocs/topics/readSqlDatabases.md Outdated
Comment thread docs/StardustDocs/topics/dataSources/Parquet.md
Comment thread docs/StardustDocs/topics/dataSources/Parquet.md Outdated
Comment thread docs/StardustDocs/topics/read.md Outdated
Comment thread docs/StardustDocs/topics/read.md Outdated
@koperagen
Copy link
Copy Markdown
Collaborator

Tables in Manage type clashes and Column type inference from CSV could be replaced with iframes

@Allex-Nik Allex-Nik added this to the Post-RC milestone May 4, 2026
@Allex-Nik Allex-Nik force-pushed the data-sources-samples-korro branch from 655d665 to 9fe3c6d Compare May 5, 2026 13:51
@Allex-Nik
Copy link
Copy Markdown
Collaborator Author

Tables in Manage type clashes and Column type inference from CSV could be replaced with iframes

I have done it in these sections (and in some others for consistency) and also added iframes for the results where it seemed useful

@Allex-Nik Allex-Nik force-pushed the data-sources-samples-korro branch from 9fe3c6d to dedd1db Compare May 8, 2026 10:38
@Allex-Nik Allex-Nik requested a review from koperagen May 8, 2026 11:19
@Allex-Nik Allex-Nik force-pushed the data-sources-samples-korro branch from dedd1db to 6e5af44 Compare May 15, 2026 08:33
@Allex-Nik Allex-Nik force-pushed the data-sources-samples-korro branch from 6e5af44 to 8034fa7 Compare May 18, 2026 14:55
@Allex-Nik Allex-Nik merged commit de65863 into master May 19, 2026
4 checks passed
@Allex-Nik Allex-Nik deleted the data-sources-samples-korro branch May 19, 2026 07:53
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.

4 participants