Break up interdependencies of command crates #9429
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Make sure that our different crates that contain commands can be compiled in parallel.
This can under certain circumstances accelerate the compilation with sufficient multithreading available.
Details
help
commands fromnu-cmd-lang
back tonu-command
nu-cmd-lang
only a dev dependency fornu-command
nu-cmd-extra
andnu-cmd-dataframe
to have a consistent api used insrc/main.rs
:get_engine_state()
nu-command
now indepedent fromnu-cmd-extra
andnu-cmd-dataframe
that are now dependencies ofnu
directly. (change to internal features)nu-command::create_default_context()
with replacement functionsFrom scratch compilation times:
just debug (dev) build and default features
before
after
User-Facing Changes
None direct, only change to compilation on multithreaded jobs expected.
Tests + Formatting
Tests that previously chose to use
nu-command
for their scope will still usenu-cmd-lang
+nu-command
(command list in the granularity at the time)