nushell/crates/nu-cmd-dataframe
Reilly Wood d5fa7b8a55
Put heavy dataframe dependencies behind feature flag (#9971)
Context from Discord:
https://discord.com/channels/601130461678272522/615962413203718156/1138694933545504819

I was working on Nu for the first time in a while and I noticed that
sometimes rust-analyzer takes a really long time to run `cargo check` on
the entire workspace. I dug in and it was checking a bunch of
dataframe-related dependencies even though the `dataframe` feature is
not built by default.

It looks like this is a regression of sorts, introduced by
https://github.com/nushell/nushell/pull/9241. Thankfully the fix is
pretty easy, we can make it so everything important in
`nu-cmd-dataframe` is only used when the `dataframe` feature is enabled.

### Impact on `cargo check --workspace`

Before this PR: 635 crates, 33.59s
After this PR: 498 crates, ~20s

(with the `mold` linker and a `cargo clean` before each run, the
relative difference for incremental checks will likely be much larger)
2023-08-09 22:36:09 -07:00
..
src Put heavy dataframe dependencies behind feature flag (#9971) 2023-08-09 22:36:09 -07:00
Cargo.toml Put heavy dataframe dependencies behind feature flag (#9971) 2023-08-09 22:36:09 -07:00
LICENSE move dataframe commands to nu-cmd-dataframe (#9241) 2023-05-19 10:56:08 -07:00