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.

Sign up for GitHub

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

Jump to bottom

More snap cleanup #39519

Merged
merged 2 commits into from
Feb 5, 2017
Merged

More snap cleanup #39519

merged 2 commits into from
Feb 5, 2017

Conversation

nagisa
Copy link
Member

@nagisa nagisa commented Feb 4, 2017

@est31 est31 mentioned this pull request Feb 4, 2017
Closed
@est31
Copy link
Member

est31 commented Feb 4, 2017

@nagisa you can also remove the ignore-stage from i128-ffi.rs and i128.rs and u128.rs. This is the only thing that I found that my PR #39521 did more than yours :)

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Feb 4, 2017

📌 Commit 206ee20 has been approved by alexcrichton

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
@frewsxcv
Rollup merge of rust-lang#39519 - nagisa:more-snap, r=alexcrichton
e0f3512 
More snap cleanup

r? @alexcrichton
@frewsxcv frewsxcv mentioned this pull request Feb 5, 2017
Closed
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
@frewsxcv
Rollup merge of rust-lang#39519 - nagisa:more-snap, r=alexcrichton
509995c 
More snap cleanup

r? @alexcrichton
@frewsxcv frewsxcv mentioned this pull request Feb 5, 2017
Closed
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 5, 2017
@frewsxcv
Rollup merge of rust-lang#39519 - nagisa:more-snap, r=alexcrichton
13b8e4b 
More snap cleanup

r? @alexcrichton
@frewsxcv frewsxcv mentioned this pull request Feb 5, 2017
Merged
bors added a commit that referenced this pull request Feb 5, 2017
@bors
Auto merge of #39567 - frewsxcv:rollup, r=frewsxcv
031c116 
Rollup of 12 pull requests

- Successful merges: #39439, #39472, #39481, #39491, #39501, #39509, #39514, #39519, #39526, #39528, #39530, #39538
- Failed merges:
@bors bors merged commit 206ee20 into rust-lang:master Feb 5, 2017
@est31 est31 mentioned this pull request Feb 7, 2017
Merged
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 7, 2017
@frewsxcv
Rollup merge of rust-lang#39604 - est31:i128_tests, r=alexcrichton
d18deea 
Unignore u128 test for stage 0,1

Even more SNAP cleanup.

Follow-up of rust-lang#39519.

Sorry, I didn't check twice.
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 7, 2017
@frewsxcv
Rollup merge of rust-lang#39604 - est31:i128_tests, r=alexcrichton
9d5dbeb 
Unignore u128 test for stage 0,1

Even more SNAP cleanup.

Follow-up of rust-lang#39519.

Sorry, I didn't check twice.
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 8, 2017
@frewsxcv
Rollup merge of rust-lang#39604 - est31:i128_tests, r=alexcrichton
ba808c3 
Unignore u128 test for stage 0,1

Even more SNAP cleanup.

Follow-up of rust-lang#39519.

Sorry, I didn't check twice.
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 9, 2017
@frewsxcv
Rollup merge of rust-lang#39604 - est31:i128_tests, r=alexcrichton
a05cc5c 
Unignore u128 test for stage 0,1

Even more SNAP cleanup.

Follow-up of rust-lang#39519.

Sorry, I didn't check twice.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@est31 est31 est31 approved these changes

Assignees

@alexcrichton alexcrichton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nagisa @est31 @alexcrichton @bors