Skip to content

Show previous definition of duplicate impl item - #33107

Merged
bors merged 1 commit into
rust-lang:masterfrom
sanxiyn:prev-impl-item
Apr 26, 2016
Merged

Show previous definition of duplicate impl item#33107
bors merged 1 commit into
rust-lang:masterfrom
sanxiyn:prev-impl-item

Conversation

@sanxiyn

@sanxiynsanxiyn commented Apr 20, 2016

Copy link
Copy Markdown
Contributor

Fix#32971.

@rust-highfive

rust-highfive commented Apr 20, 2016

Copy link
Copy Markdown
Contributor

r? @jroesch

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

@eddyb

eddyb commented Apr 21, 2016

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Apr 21, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit f532685 has been approved by eddyb

@bors

bors commented Apr 22, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f532685 with merge e8d44e5...

@bors

bors commented Apr 22, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-msvc-64-opt-mir

@alexcrichton

alexcrichton commented Apr 22, 2016

Copy link
Copy Markdown
Member

@bors: retry

On Fri, Apr 22, 2016 at 8:39 AM, bors notifications@github.com wrote:

[image: 💔] Test failed - auto-win-msvc-64-opt-mir
http://buildbot.rust-lang.org/builders/auto-win-msvc-64-opt-mir/builds/410

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#33107 (comment)

@bors

bors commented Apr 24, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f532685 with merge 9687604...

@bors

bors commented Apr 24, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-gnu-64-opt

@alexcrichton

alexcrichton commented Apr 24, 2016

Copy link
Copy Markdown
Member

@bors: retry

On Sun, Apr 24, 2016 at 4:34 AM, bors notifications@github.com wrote:

[image: 💔] Test failed - auto-win-gnu-64-opt
http://buildbot.rust-lang.org/builders/auto-win-gnu-64-opt/builds/3925

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#33107 (comment)

@bors

bors commented Apr 25, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f532685 with merge af43dbe...

@bors

bors commented Apr 25, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@alexcrichton

alexcrichton commented Apr 25, 2016

Copy link
Copy Markdown
Member

@bors: retry

Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 25, 2016
@Manishearth
Rollup merge ofrust-lang#33107- sanxiyn:prev-impl-item, r=eddyb
1bc30a5
Show previous definition of duplicate impl item
Fixrust-lang#32971.
@ManishearthManishearth mentioned this pull request Apr 25, 2016
Merged
bors added a commit that referenced this pull request Apr 25, 2016
@bors
Auto merge of#33204- Manishearth:rollup, r=Manishearth
149cf77
Rollup of 7 pull requests
- Successful merges: #33107, #33133, #33160, #33167, #33194, #33196, #33200
- Failed merges:
bors added a commit that referenced this pull request Apr 26, 2016
@bors
Auto merge of#33204- Manishearth:rollup, r=Manishearth
41028de
Rollup of 7 pull requests
- Successful merges: #33107, #33133, #33160, #33167, #33194, #33196, #33200
- Failed merges:
@bors
bors merged commit f532685 into rust-lang:masterApr 26, 2016
@sanxiyn
sanxiyn deleted the prev-impl-item branch April 26, 2016 07:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jroeschjroesch

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@sanxiyn@rust-highfive@eddyb@bors@alexcrichton@jroesch