Skip to content

Remove/rework some redundant .cwd(p.root()) in tests#5987

Merged
bors merged 4 commits into
rust-lang:masterfrom
dwijnand:remove-cwd-root
Sep 6, 2018
Merged

Remove/rework some redundant .cwd(p.root()) in tests#5987
bors merged 4 commits into
rust-lang:masterfrom
dwijnand:remove-cwd-root

Conversation

@dwijnand

@dwijnand dwijnand commented Sep 6, 2018

Copy link
Copy Markdown
Contributor

Just a little trim to the tests.

@rust-highfive

Copy link
Copy Markdown

r? @alexcrichton

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

@dwijnand dwijnand changed the title Remove/reworksome redundant .cwd(p.root()) in tests Remove/rework some redundant .cwd(p.root()) in tests Sep 6, 2018
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Sep 6, 2018

Copy link
Copy Markdown
Contributor

📌 Commit 3b786a0 has been approved by alexcrichton

bors added a commit that referenced this pull request Sep 6, 2018
Remove/rework some redundant .cwd(p.root()) in tests

Just a little trim to the tests.
@bors

bors commented Sep 6, 2018

Copy link
Copy Markdown
Contributor

⌛ Testing commit 3b786a0 with merge a39c2a3...

@bors

bors commented Sep 6, 2018

Copy link
Copy Markdown
Contributor

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

@bors bors merged commit 3b786a0 into rust-lang:master Sep 6, 2018
@dwijnand dwijnand deleted the remove-cwd-root branch September 6, 2018 18:56
@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.

5 participants