Skip to content

Conversation

@dwijnand
Copy link
Contributor

@dwijnand dwijnand commented Sep 4, 2018

@rust-highfive
Copy link

r? @matklad

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

@dwijnand
Copy link
Contributor Author

dwijnand commented Sep 4, 2018

If this is accepted I think https://github.com/rust-lang/cargo/wiki/Cargo's-architecture should be deleted, or emptied and link to ARCHITECTURE.md.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Sep 4, 2018

📌 Commit 37a64d4 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Sep 4, 2018

⌛ Testing commit 37a64d4 with merge e7f00b3...

bors added a commit that referenced this pull request Sep 4, 2018
Migrate some architecture notes from the wiki

Merge https://github.com/rust-lang/cargo/wiki/Cargo's-architecture into
the ARCHITECTURE.md in the repo.

Fixes #5528
@bors
Copy link
Contributor

bors commented Sep 4, 2018

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

@bors bors merged commit 37a64d4 into rust-lang:master Sep 4, 2018
@dwijnand dwijnand deleted the one-architecture branch September 4, 2018 16:01
@ehuss ehuss added this to the 1.30.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.

ARCHITECTURE.md vs Cargo's-Architecture.md

6 participants