Skip to content

feat: optimize library API when using csvlens as a library, dropping clap dependency#118

Merged
YS-L merged 5 commits intoYS-L:mainfrom
jqnatividad:116-clap-optional-dependency-cli-feature
Jan 11, 2025
Merged

feat: optimize library API when using csvlens as a library, dropping clap dependency#118
YS-L merged 5 commits intoYS-L:mainfrom
jqnatividad:116-clap-optional-dependency-cli-feature

Conversation

@jqnatividad
Copy link
Copy Markdown
Contributor

  • added run_csvlens_with_options for a more ergonomic API interface for library usage
  • bumped dependencies while at it, except arrow, as Polars is still on arrow 53

resolves #116

though I did hold back arrow to v53 (coz another library I was using - Polars, was still at v53)
for backwards compatibility, we retain the old `run_csvlens` library for just parsing the input csv and introduce a new `run_csvlens_with_options` to pass csvlens parameters as a struct
as it turns out Polars is using its own fork of arrow2
@YS-L
Copy link
Copy Markdown
Owner

YS-L commented Jan 11, 2025

Thanks @jqnatividad !

@YS-L YS-L merged commit f4d2e47 into YS-L:main Jan 11, 2025
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.

feature: make clap an optional dependency when being used as a library

2 participants