Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRAFT: Start adding support for workspace #804

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Its-Just-Nans
Copy link

Hello

Here is a little piece of draft code that should handle workspace

Note that WorkspaceConfig and CargoMetadata are very similar, maybe I should merge them ?

I am open for ideas and help :)

Related to

usage for test

cargo build
cd examples/workplace
../../target/debug/trunk serve

@@ -34,7 +34,9 @@ console = "0.15"
directories = "5"
dunce = "1"
flate2 = "1"
futures-util = { version = "0.3", default-features = false, features = ["sink"] }
futures-util = { version = "0.3", default-features = false, features = [
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please keep the one line format, as this allows to easily sort the dependencies alphabetically.

@ctron ctron marked this pull request as draft May 31, 2024 06:47
@ctron
Copy link
Collaborator

ctron commented May 31, 2024

Thanks for the PR!.

I am not sure I fully understand what the goal is? Maybe you can explain a bit what you have in mind.

Btw, you should be able run this using:

cargo run -- --config examples/workplace

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.

2 participants