Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests for E-needstest issues #19780

Merged
merged 1 commit into from Dec 18, 2014
Merged

Add tests for E-needstest issues #19780

merged 1 commit into from Dec 18, 2014

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