Skip to content

Move the man directory to a subdirectory#44413

Merged
bors merged 1 commit intorust-lang:masterfrom
est31:move_man
Sep 12, 2017
Merged

Move the man directory to a subdirectory#44413
bors merged 1 commit intorust-lang:masterfrom
est31:move_man

Conversation

@est31
Copy link
Member

@est31 est31 commented Sep 8, 2017

There is no reason it should be in the top directory.

There is no reason it should be in the top directory.
@rust-highfive
Copy link
Contributor

r? @aturon

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

@est31
Copy link
Member Author

est31 commented Sep 8, 2017

r? @steveklabnik this is a docs issue. Also cc @GuillaumeGomez @QuietMisdreavus @frewsxcv the docs team

@rust-highfive rust-highfive assigned steveklabnik and unassigned aturon Sep 8, 2017
@steveklabnik
Copy link
Contributor

It's really @rust-lang/compiler, IMO; the layout of the repo is up to them.

I agree that it being at the top level is unneeded but I don't have strong feelings.

@GuillaumeGomez
Copy link
Member

Well, seems good to me.

@est31
Copy link
Member Author

est31 commented Sep 10, 2017

r? @nikomatsakis

@carols10cents carols10cents added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 11, 2017
@nikomatsakis
Copy link
Contributor

@bors r+ -- sure.

@bors
Copy link
Collaborator

bors commented Sep 11, 2017

📌 Commit d587558 has been approved by nikomatsakis

@bors
Copy link
Collaborator

bors commented Sep 12, 2017

⌛ Testing commit d587558 with merge a7b7dce...

bors added a commit that referenced this pull request Sep 12, 2017
Move the man directory to a subdirectory

There is no reason it should be in the top directory.
@bors
Copy link
Collaborator

bors commented Sep 12, 2017

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

@bors bors merged commit d587558 into rust-lang:master Sep 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants