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

Rollup of stuff (take 2) #21698

Merged
merged 47 commits into from
Jan 28, 2015
Merged

Rollup of stuff (take 2) #21698

merged 47 commits into from
Jan 28, 2015

Conversation

Manishearth
Copy link
Member

This should work now.

GuillaumeGomez and others added 30 commits January 24, 2015 13:58
Removes min-width rule from body so that the page does not scroll
horizontally on mobile displays; hides out-of-band information.
Lifetime elision with two input references is not clear.
Closes rust-lang#21284
when saving .rs files under vim
do not fail to run the syntax checker
error: Unrecognized option: 'parse-only'.

due to this commit
953f294
which removed the deprecated flag --parse-only
While waiting on some builds I started cleaning up the various python bits and pieces.

I'm going to keep poking, want to ping me before the next rollup?
…r=alexcrichton

Cleanup mention of uint, use usize/us instead. This includes the example `println!("{}", 2u * x);`
@eddyb
Copy link
Member

eddyb commented Jan 27, 2015

@Manishearth You could have just fixed it (see my comment on his PR).

@Manishearth
Copy link
Member Author

@eddyb History rewrites with merge commits are a bit harder to do and the merge commits tend to disappear.

@emberian
Copy link
Member

@bors r+ 249c29f

@bors
Copy link
Collaborator

bors commented Jan 28, 2015

⌛ Testing commit 249c29f with merge ee5fb94...

bors added a commit that referenced this pull request Jan 28, 2015
@bors
Copy link
Collaborator

bors commented Jan 28, 2015

💔 Test failed - auto-win-64-opt

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Jan 28, 2015

☔ Merge conflict

@richo
Copy link
Contributor

richo commented Jan 28, 2015

oof :(

@Manishearth
Copy link
Member Author

Oh, no. @alexcrichton I seem to have bad luck rebasing these -- what sort of flags besides --preserve-merges do I feed to git rebase?

@alexcrichton
Copy link
Member

I normally actually just run git merge origin/master as rebasing all the PRs is too much of a pain. Failing that git rebase -p should do the trick (to preserve merge commits)

@alexcrichton
Copy link
Member

@bors: r+ c7c7d64 p=10

@bors
Copy link
Collaborator

bors commented Jan 28, 2015

⌛ Testing commit c7c7d64 with merge 9cc35d7...

Conflicts:
	src/libcollections/slice.rs
	src/libcore/nonzero.rs
	src/libcore/ops.rs
@alexcrichton
Copy link
Member

@bors: r+ c709ed2 rollup p=10

@alexcrichton
Copy link
Member

@bors: p=10

@alexcrichton
Copy link
Member

@bors: rollup-

bors added a commit that referenced this pull request Jan 28, 2015
@bors
Copy link
Collaborator

bors commented Jan 28, 2015

⌛ Testing commit c709ed2 with merge ee0be3b...

@bors bors merged commit c709ed2 into rust-lang:master Jan 28, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.