Skip to content

More snap cleanup #39519

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
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
Collaborator

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.
# for free to join this conversation on GitHub. Already have an account? # 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