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

Document that read_manifest command is deprecated #3150

Merged
merged 1 commit into from
Oct 3, 2016

Conversation

matklad
Copy link
Member

@matklad matklad commented Oct 2, 2016

I believe we intended to deprecate read_manifest command. I am not sure what a deprecation process for Cargo commands should be, but I guess it should involve mentioning somewhere that the command is deprecated :)

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 3, 2016

📌 Commit fdbca1c has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Oct 3, 2016

⌛ Testing commit fdbca1c with merge 6428537...

bors added a commit that referenced this pull request Oct 3, 2016
Document that read_manifest command is deprecated

I believe we intended to deprecate read_manifest command. I am not sure what a deprecation process for Cargo commands should be, but I guess it should involve mentioning somewhere that the command is deprecated :)
@bors
Copy link
Contributor

bors commented Oct 3, 2016

☀️ Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 6428537 to master...

@bors bors merged commit fdbca1c into rust-lang:master Oct 3, 2016
@matklad matklad deleted the deprecate-read-manifest branch December 14, 2016 15:27
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.

4 participants