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

[docs/manifest] Add Virtual Manifest section #4376

Merged
merged 1 commit into from
Aug 8, 2017
Merged

Conversation

behnam
Copy link
Contributor

@behnam behnam commented Aug 7, 2017

Current documentation does not mention Virtual Manifests at all, which
can be confusing because they appear in error messages.

Here we add the definition, and provide a hint about --all option for
most cargo commands, which allow the command to work in the way most
probably expected.

Fixes #3526

Current documentation does not mention Virtual Manifests at all, which
can be confusing because they appear in error messages.

Here we add the definition, and provide a hint about `--all` option for
most cargo commands, which allow the command to work in the way most
probably expected.

Fixes <rust-lang#3526>
@behnam
Copy link
Contributor Author

behnam commented Aug 7, 2017

r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Aug 7, 2017

📌 Commit 3f5ca7b has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Aug 8, 2017

⌛ Testing commit 3f5ca7b with merge fe56d43...

bors added a commit that referenced this pull request Aug 8, 2017
[docs/manifest] Add Virtual Manifest section

Current documentation does not mention Virtual Manifests at all, which
can be confusing because they appear in error messages.

Here we add the definition, and provide a hint about `--all` option for
most cargo commands, which allow the command to work in the way most
probably expected.

Fixes <#3526>
@bors
Copy link
Contributor

bors commented Aug 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing fe56d43 to master...

@bors bors merged commit 3f5ca7b into rust-lang:master Aug 8, 2017
@behnam behnam deleted the virtual branch August 8, 2017 01:55
@ehuss ehuss added this to the 1.21.0 milestone Feb 6, 2022
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.

(Virtual?) manifest with both package and workspace sections is accepted
4 participants