Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Dec 12, 2014

Closes #5988.
Closes #10176.
Closes #10456.
Closes #12744.
Closes #13264.
Closes #13324.
Closes #14182.
Closes #15381.
Closes #15444.
Closes #15480.
Closes #15756.
Closes #16822.
Closes #16966.
Closes #17351.
Closes #17503.
Closes #17545.
Closes #17771.
Closes #17816.
Closes #17897.
Closes #17905.
Closes #18188.
Closes #18232.
Closes #18345.
Closes #18389.
Closes #18400.
Closes #18502.
Closes #18611.
Closes #18783.
Closes #19009.
Closes #19081.
Closes #19098.
Closes #19127.
Closes #19135.

@ghost
Copy link
Author

ghost commented Dec 12, 2014

I tried to make sure they're testing what they're supposed to and they don't overlap. I'll close the remaining ones manually.

@alexcrichton
Copy link
Member

Whoa, that is quite the list of issues, thanks so much @jakub-! It appears there's a tidy error, but other than that r=me

@brson
Copy link
Contributor

brson commented Dec 12, 2014

Wow.

@steveklabnik
Copy link
Member

One ❤️ just isn't enough.

❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️
❤️

(I think that matches the issue count, haha!)

@brson
Copy link
Contributor

brson commented Dec 14, 2014

@barosl
Copy link
Contributor

barosl commented Dec 14, 2014

It seems that those two failed tests are missing the aux-build directive.

bors added a commit that referenced this pull request Dec 14, 2014
Add tests for E-needstest issues

Reviewed-by: alexcrichton
@ghost
Copy link
Author

ghost commented Dec 17, 2014

@barosl Indeed, thanks :)

bors added a commit that referenced this pull request Dec 18, 2014
Closes #5988.
Closes #10176.
Closes #10456.
Closes #12744.
Closes #13264.
Closes #13324.
Closes #14182.
Closes #15381.
Closes #15444.
Closes #15480.
Closes #15756.
Closes #16822.
Closes #16966.
Closes #17351.
Closes #17503.
Closes #17545.
Closes #17771.
Closes #17816.
Closes #17897.
Closes #17905.
Closes #18188.
Closes #18232.
Closes #18345.
Closes #18389.
Closes #18400.
Closes #18502.
Closes #18611.
Closes #18783.
Closes #19009.
Closes #19081.
Closes #19098.
Closes #19127.
Closes #19135.
@bors bors merged commit 62d80df into rust-lang:master Dec 18, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment