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

[doc/crates-io] Add text about packaging tests/benches #4272

Merged
merged 1 commit into from
Jul 12, 2017

Conversation

behnam
Copy link
Contributor

@behnam behnam commented Jul 11, 2017

Fixes #4262

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Jul 11, 2017

📌 Commit 4c1d216 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 11, 2017

⌛ Testing commit 4c1d2169de378fb869abdd05025f57c4fb342156 with merge e0db677d7c850ea7d70c42e0717f1f4f57c2033e...

@behnam
Copy link
Contributor Author

behnam commented Jul 11, 2017

I noticed a typo and fixed it in da96699 . Not sure if @bors is going to pick up the new commit.

@behnam
Copy link
Contributor Author

behnam commented Jul 11, 2017

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Jul 11, 2017

@behnam: 🔑 Insufficient privileges: Not in reviewers

@matklad
Copy link
Member

matklad commented Jul 12, 2017

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Jul 12, 2017

📌 Commit da96699 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 12, 2017

⌛ Testing commit da96699 with merge 74e5421...

bors added a commit that referenced this pull request Jul 12, 2017
[doc/crates-io] Add text about packaging tests/benches

Fixes <#4262>
@bors
Copy link
Contributor

bors commented Jul 12, 2017

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

@bors bors merged commit da96699 into rust-lang:master Jul 12, 2017
@bors bors mentioned this pull request Jul 12, 2017
@behnam behnam deleted the doc branch July 13, 2017 02:42
@ehuss ehuss added this to the 1.20.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.

Default exclude list for cargo --new
5 participants